-
Notifications
You must be signed in to change notification settings - Fork 5
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
Postpone Metamask integration #61
Comments
Whats the difference to #31 ? |
So, in the UI there is 3 states, we have to "postpone" then. How it could lookThis includes three states:
All three are postponed to "buy & send" screen. @artyomignatyev what do you think? Do you think there should be a popup, or should this be fullscreen? |
I think we also really need a better "network is not supported" network screen, anyhow knows a dapp thats follows best practice here? |
FYI, you don't need to ask a user to open MetaMask to log in anymore, thanks to the new method |
Thanks for the reminder @danfinlay . We are using @artyomignatyev i updated the figma screenshots above. Feedback welcome ;) |
This is a ✨ Cryptoxmas.xyz bounty✨
Every bounty DAI paid out will be matched as a donation to this Venezuelan Giveth Campaign
Userstory
As a user I want to be able to view Christmas Cards before being asked to login via metamask,
so that I can see the selection of images from a normal web2 browser.
Remove need to login to browse tokens
Requires
UI
Will be proposed by @seichris & @artyomignatyev
The text was updated successfully, but these errors were encountered: