-
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: don't reject when signing #2666
Conversation
Branch preview⏳ Deploying a preview site... |
ESLint Summary View Full Report
Report generated by eslint-plus-action |
Coverage report
Test suite run success1043 tests passing in 141 suites. Report generated by 🧪jest coverage report action from 30890fb |
src/components/tx-flow/index.tsx
Outdated
return newTxFlow | ||
}) | ||
(newTxFlow: TxModalContextType['txFlow'], onClose?: () => void, newShouldWarn?: boolean) => { | ||
// If flow is open user opens a different one, show confirmation dialog if required |
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.
Is there an and
missing?
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 c961fbf.
What it solves
Resolves successful signing requests rejecting in dApps
How this PR fixes it
The
USER_QUIT
event is nly dispatched to the dApp if a warning requirement was set and is accepted when closing the flow, ergo not when a signature request succeeds and closes the flow.How to test it
Observe successful swaps on Aave, CoWSwap, Curve and Sushi (on 1/n Safes).
Checklist