FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

fix: update protocol version and rename properties in initialization request by anuraj · Pull Request #1878 · modelcontextprotocol/csharp-sdk · GitHub

Repository navigation

fix: update protocol version and rename properties in initialization request - #1878

Open
anuraj wants to merge 2 commits into
modelcontextprotocol:mainfrom
anuraj:main
Open

anuraj wants to merge 2 commits into
modelcontextprotocol:mainfrom
anuraj:main

Conversation

anuraj commented Sep 15, 2026

Copy link
Copy Markdown

Summary

The current MCP UI Sample - WeatherAppServer is not working properly. It was showing an error and not displaying the data.

Motivation and Context

This change will fix the issue and weather app server will work properly

How Has This Been Tested?

The sample is tested with MCP Inspector and VS Code - GitHub Copilot.

Breaking Changes

No

Additional context

NA

Copilot AI lite review requested due to automatic review settings September 15, 2026 16:18

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

Note

This error may be related to your runner configuration. You can now configure runners for Copilot code review separately from Copilot cloud agent by creating a copilot-code-review.yml file with your setup steps. Read the docs for details.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

🟢 Approval recommended

No unresolved review comments remain.

Review details
  • Files reviewed: 1/1 changed files
  • Comments generated: 0 new
  • Review effort level: Lite

This branch has not been deployed

No deployments
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
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


Back | FazBrowse Home | New Git URL