Skip to content

fix(methodObject): remove result as required#158

Merged
shanejonas merged 3 commits intomasterfrom
fix/methodobject-remove-result-req
Feb 1, 2023
Merged

fix(methodObject): remove result as required#158
shanejonas merged 3 commits intomasterfrom
fix/methodobject-remove-result-req

Conversation

@shanejonas
Copy link
Member

@shanejonas shanejonas self-assigned this Oct 22, 2019
Copy link
Member

@BelfordZ BelfordZ left a comment

Choose a reason for hiding this comment

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

Please follow up in the issue in the spec.

@BelfordZ
Copy link
Member

please continue discussion here: open-rpc/spec#230

@BelfordZ BelfordZ closed this Nov 19, 2019
meowsbits pushed a commit to meowsbits/meta-schema that referenced this pull request Mar 26, 2020
feat(MethodCallValidator): allow rpc discover
@shanejonas shanejonas reopened this Jan 27, 2023
@shanejonas
Copy link
Member Author

reopening this again around the notification definition discussion: open-rpc/spec#230

@shanejonas shanejonas merged commit af57ceb into master Feb 1, 2023
@shanejonas shanejonas deleted the fix/methodobject-remove-result-req branch February 1, 2023 04:06
@BelfordZ
Copy link
Member

BelfordZ commented Feb 1, 2023

🎉 This PR is included in version 1.14.3 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

How to define a notification ?

3 participants