Skip to content
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

Manage MetaMask connection in store instead of component #143

Open
annybs opened this issue Jan 19, 2022 · 0 comments
Open

Manage MetaMask connection in store instead of component #143

annybs opened this issue Jan 19, 2022 · 0 comments

Comments

@annybs
Copy link
Member

annybs commented Jan 19, 2022

Similar to #142 -

At the moment, the Deposit modal contains all state pertaining to the page's MetaMask connection. This state can be lost while navigating between pages.

Moving the MetaMask connection state (and associated values) to the global store will keep the connection alive, and also make it easier to validate & respond to changes independently of individual components. We could consider adding UI elements specifically for managing connection status, e.g. status banner, a button to forcibly disconnect MetaMask.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant