-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
fix!: remove unnecessary nonce from message gql queries #3298
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
1 Skipped Deployment
|
CodSpeed Performance ReportMerging #3298 will not alter performanceComparing Summary
|
Co-authored-by: Nedim Salkić <nedim.salkic@fuel.sh>
This can be done at a later date - this optimization doesn't provide (much value). |
Pull request was closed
Re-opening this so that we may re-introduce breaking changes at a later more appropriate date. |
Putting this as a draft since it's breaking |
Coverage Report:
Changed Files:
|
Release notes
In this release, we:
getMessageByNonce
provider endpointSummary
The
nonce
returned from thegql
query was removed from themessageFragment
operation as the parameter can actually be re-used in the functiongetMessageByNonce
and a separate fragment was created forgetMessage
Breaking Changes
nonce
property fromProvider.operations.getMessageByNonce()
. This can still be retrieved byProvider.getMessageByNonce()
.Checklist