-
Notifications
You must be signed in to change notification settings - Fork 619
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
feat: disable request queue #4447
Conversation
seems like there's also a bug fix with |
I'm testing what might be causing the network drops |
# Conflicts: # package-lock.json # packages/sign-client/package.json
# Conflicts: # packages/sign-client/src/controllers/engine.ts
type SignConfig = { | ||
disableRequestQueue?: boolean; | ||
}; |
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.
nitpick, optional: shouldn't this be in types?
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.
what do you mean? It is in packages/types/src/sign-client/client.ts
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.
Thanks for the solid test coverage on the new option 🙏
…/walletconnect-monorepo into feat/disable-request-queue
Description
Added new init param
signConfig
with initialdisableRequestQueue
parameter. This option allows wallets to disable the session request queue if they wish.Additionally, added a few improvements to connection handling because if several days long
socket hang up
issues in CI such asping
timers when connection is closed intentionallyType of change
How has this been tested?
tests
canary -
2.12.2-req-q.0
Checklist
Additional Information (Optional)
Please include any additional information that may be useful for the reviewer.