Skip to content
This repository has been archived by the owner. It is now read-only.

Wallet selection #176

merged 41 commits into from Jul 17, 2019

Wallet selection #176

merged 41 commits into from Jul 17, 2019


Copy link

@jernejpregelj jernejpregelj commented Jul 9, 2019

Default to burner wallet and allow to switch between burner wallet and MetaMask. Essentially upgrades non-Web3 users into Web3 users. UI flow outlined in #169 (comment).

And more:

  • detect installed market network from Brizo, use it to adapt copy throughout UI
  • improvements to Web3message output, e.g. shown on About page

New default minimal variant of Web3message. Will be appended with warnings if there are any, as seen in extended version further down.

Most of our users will come with non-Web3-capable browsers so this is what they'll see:

Screen Shot 2019-07-11 at 14 55 24

In Web3-capable browsers we show a select wallet action:

Screen Shot 2019-07-10 at 18 37 17

Screen Shot 2019-07-10 at 18 44 42

Wallet selection modal:

Screen Shot 2019-07-10 at 21 23 00

Extended version on About page:

Screen Shot 2019-07-10 at 18 36 12

Screen Shot 2019-07-10 at 20 26 44

Screen Shot 2019-07-10 at 20 24 23

Screen Shot 2019-07-10 at 17 31 13

@jernejpregelj jernejpregelj requested a review from Jul 9, 2019
@jernejpregelj jernejpregelj requested a review from as a code owner Jul 9, 2019
@kremalicious kremalicious force-pushed the feature/wallets branch 2 times, most recently from 273bc0a to 7dd34e8 Jul 9, 2019
@kremalicious kremalicious changed the title Feature/wallets Wallet selection Jul 9, 2019
Copy link

@kremalicious kremalicious commented Jul 9, 2019

@jernejpregelj note: already worked on this and rebased, did some rebase fixes and force pushed. So branch needs to be checked out new from remote

Copy link

@kremalicious kremalicious left a comment



@jernejpregelj jernejpregelj merged commit 599e183 into master Jul 17, 2019
3 checks passed
@kremalicious kremalicious deleted the feature/wallets branch Jul 17, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
None yet
None yet
Linked issues

Successfully merging this pull request may close these issues.

2 participants