Skip to content

doc: Add OnPermissionRequest handler to .NET / Getting Started code#957

Merged
SteveSandersonMS merged 1 commit intogithub:mainfrom
jongalloway:patch-1
Mar 30, 2026
Merged

doc: Add OnPermissionRequest handler to .NET / Getting Started code#957
SteveSandersonMS merged 1 commit intogithub:mainfrom
jongalloway:patch-1

Conversation

@jongalloway
Copy link
Copy Markdown
Contributor

Added OnPermissionRequest handler to session configuration for .NET code in Getting Started guide.

Documentation improvements:

  • Added OnPermissionRequest = PermissionHandler.ApproveAll to all SessionConfig examples to demonstrate automatic approval of permission requests in sample code. [1] [2] [3] [4]

Added OnPermissionRequest handler to session configuration for .NET code in Getting Started guide.
@jongalloway jongalloway requested a review from a team as a code owner March 30, 2026 04:40
Copilot AI review requested due to automatic review settings March 30, 2026 04:40
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Updates the .NET “Getting Started” documentation examples to include an OnPermissionRequest handler so the sample code auto-approves permission prompts during execution.

Changes:

  • Updated the basic .NET quickstart snippet to configure SessionConfig.OnPermissionRequest.
  • Added OnPermissionRequest = PermissionHandler.ApproveAll to the .NET streaming example.
  • Added OnPermissionRequest = PermissionHandler.ApproveAll to the .NET tool and interactive assistant examples.

@jongalloway jongalloway changed the title Fix Getting Started .NET code for OnPermissionRequest handler doc: Add OnPermissionRequest handler to .NET / Getting Started code Mar 30, 2026
@SteveSandersonMS SteveSandersonMS added this pull request to the merge queue Mar 30, 2026
Merged via the queue into github:main with commit e3638da Mar 30, 2026
7 of 8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants