-
Notifications
You must be signed in to change notification settings - Fork 363
fix: Redirect user to next available safe after removing current safe #3424
Conversation
CLA Assistant Lite All Contributors have signed the CLA. |
ESLint Summary View Full Report
Report generated by eslint-plus-action |
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.
Looking good, but I don't agree with switching to a different network.
E2E Tests Failed Failed tests:
|
ESLint Summary View Full Report
Report generated by eslint-plus-action |
Looks good, The safe goes to the 1st safe in the network when another is deleted. |
What it solves
Resolves #3031
How this PR fixes it
Similar to how its handled on mobile, when deleting the current safe the user gets redirected to:
Open Questions
Could opening another safe automatically impact UX in a negative way? cc @liliiaorlenko
How to test it