-
Notifications
You must be signed in to change notification settings - Fork 411
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: emit chainChanged
first
#2661
Conversation
Branch preview✅ Deploy successful! https://wc_chain_alignment--walletweb.review-wallet-web.5afe.dev |
ESLint Summary View Full Report
Report generated by eslint-plus-action |
Coverage report
Show files with reduced coverage 🔻
Test suite run success1026 tests passing in 141 suites. Report generated by 🧪jest coverage report action from c47e37a |
@@ -344,36 +344,36 @@ describe('WalletConnectWallet', () => { | |||
expect(emitSessionEventSpy).toHaveBeenCalledTimes(2) | |||
}) | |||
|
|||
it('should call emitSessionEvent with the correct parameters', async () => { | |||
it.only('should call emitSessionEvent with the correct parameters', async () => { |
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.
Don't forget the only.
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.
Addressed in 09db2d4.
What it solves
Resolves dApp chain alignment
How this PR fixes it
The
chainChanged
event is not emitted before the `accountsChanged event when updating a session.How to test it
Connect to Aura Finance and observe the initial chain matching that of the Safe, as well as changing with Safe switches across chains.
Checklist