-
Notifications
You must be signed in to change notification settings - Fork 75
feat: add message field to ProgressNotification according to 2025-03-26 spec #69
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
feat: add message field to ProgressNotification according to 2025-03-26 spec #69
Conversation
I have a question why do we have ProgressNotification and Notification? The only difference is the _meta and method fields. |
Carefull about your cctv nvr use in eseecloud |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.
Working fine
Adrian
…On Thu, Apr 10, 2025 at 2:21 AM Leonid Stashevsky ***@***.***> wrote:
***@***.**** approved this pull request.
—
Reply to this email directly, view it on GitHub
<#69 (review)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/BJVKREFQO3XBYB3YG3RKY7T2YYS2LAVCNFSM6AAAAAB2X4ANAGVHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMZDONJVHAYDANRTGE>
.
You are receiving this because you commented.Message ID:
***@***.***>
|
You know what you guys can move the token anyway so just have it and good bless |
Motivation and Context
This PR adds message field to ProgressNotification to provide descriptive status updates
How Has This Been Tested?
NA
Breaking Changes
NA
Types of changes
Checklist
Additional context
References: https://github.com/modelcontextprotocol/modelcontextprotocol/blob/main/schema/2025-03-26/schema.ts#L294