Goals

• Simplify the current wallet
• Implement new UX changes
• Integrate with /analytics page to calculate earnings & payout

Milestones

Milestone	Link
Wallet Upgrades	https://gitlab.com/groups/minds/-/milestones/62

Merge Requests & Branches

TBD

Mockups

Responsiveness

Three indicative layouts for small, medium and large screens. Resources not currently available to provide responsive layouts for all the screens, so it will be at the developer's discretion to connect the dots. A CR pass can be used to identify and resolve any responsive issues.

• [Large] https://invis.io/KWU1X9IEB2U#/396034060_General_Wallet_Large
• [Medium] https://invis.io/KWU1X9IEB2U#/396034059_General_Wallet_Medium
• [Small] https://invis.io/KWU1X9IEB2U#/396034058_General_Wallet_Small

Form Validation

Client-side, inline form validation should be run on all form inputs.

• [General form validation concept] https://invis.io/KWU1X9IEB2U#/395155245_General_Form_Validation

Notifications

Any form submission to be followed up with a notification. e.g. On-Chain address updated, 100 tokens transferred to on-chain etc. It's important interactions/transactions are followed up with a confirmation message.

• [Designs TBC]

Tokens

• [Overview - Pre-setup. Unverified phone number, no on-chain address] https://invis.io/KWU1X9IEB2U#/396032646_Wallet-_Tokens-_Overview-_Unverified
• [Overview - Phone verified. On-chain address not setup] https://invis.io/KWU1X9IEB2U#/396032645_Wallet-_Tokens-_Overview-_Phone_Verified_Only
• [Phone verification modal] https://invis.io/KWU1X9IEB2U#/396032644_Wallet-_Tokens-_Overview-_Verify_Phone_Modal
• [Phone verification modal - enter code] - https://invis.io/KWU1X9IEB2U#/396078538_Wallet-_Tokens-_Overview-_Verify_Phone_-code-_Modal
• [Phone verification error] - https://invis.io/KWU1X9IEB2U#/396078537_Wallet-_Tokens-_Overview-_Verify_Phone_-code-_Modal_Error
• [Overview - all setup] https://invis.io/KWU1X9IEB2U#/396032647_Wallet-_Tokens-_Overview
• [Transactions] https://invis.io/KWU1X9IEB2U#/396032643_Wallet-_Tokens-_Transactions
• [Settings - OnChain not setup] - https://invis.io/KWU1X9IEB2U#/396032631_Wallet-_Tokens-_Settings
• [Settings - OnChain setup] - https://invis.io/KWU1X9IEB2U#/396035081_Wallet-_Tokens-_Settings_-with_On-Chain_Address-
• [On-Chain transfer modal] - https://invis.io/KWU1X9IEB2U#/396448900_Wallet-_Tokens-_Overview-_On-Chain_Transfer

USD

• [Transactions. Bank A/C not setup - default screen] https://invis.io/KWU1X9IEB2U#/396037675_Wallet-_USD-_Transactions_-Bank_Not_Setup-
• [Transactions. Bank A/C setup] https://invis.io/KWU1X9IEB2U#/396037676_Wallet-_USD-_Transactions
• [Settings - Bank A/C not setup] https://invis.io/KWU1X9IEB2U#/396037674_Wallet-_USD-_Settings_-no_Bank_Info-
• [Settings - Bank A/C form] https://invis.io/KWU1X9IEB2U#/396037672_Wallet-_USD-_Settings-_Bank_Details_Form
• [Settings - Bank A/C setup] https://invis.io/KWU1X9IEB2U#/396037673_Wallet-_USD-_Settings_-with_Bank_Info-
• [Settings - provide custom on-chain address] https://invis.io/KWU1X9IEB2U#/396453407_Wallet-_Token-_Settings_-_Update_Address
• [Settings - provide private key] https://invis.io/KWU1X9IEB2U#/396454946_Wallet-_Token-_Settings_-_Download_Key

ETH

• [Address not setup] https://invis.io/KWU1X9IEB2U#/396032640_Wallet-_Ether-_Onboard
• [Address form] https://invis.io/KWU1X9IEB2U#/396032636_Wallet-_Ether-_Form
• [Address setup] https://invis.io/KWU1X9IEB2U#/396032639_Wallet-_Ether-_Setup

BTC

• [Address not setup] https://invis.io/KWU1X9IEB2U#/396032638_Wallet-_BTC-_Onboard
• [Address form] https://invis.io/KWU1X9IEB2U#/396032637_Wallet-_BTC-_Form
• [Address setup] https://invis.io/KWU1X9IEB2U#/396032635_Wallet-_BTC-_Setup

Developers

Developer	Focus
@omadrid	frontend
@michaelfbradley	UX&D
TBC	mobile

changed the description

changed the description

Please see below latest wireframes for wallet. Note, this is still a WIP.

Wallet - Tokens [Desktop]

• https://invis.io/KWU1X9IEB2U#/386167821_Wallet-_Tokens-_General
• https://invis.io/KWU1X9IEB2U#/386167819_Wallet-_Tokens-_General_Copy
• https://invis.io/KWU1X9IEB2U#/386167817_Wallet-_Tokens-_Settings
• https://invis.io/KWU1X9IEB2U#/386167818_Wallet-_USD

Wallet - USD [Desktop]

• https://invis.io/KWU1X9IEB2U#/386167820_Wallet-_USD_General

cc @markeharding

changed the description

added issue front#1957 (closed) from epic &74 (closed)

changed title from New Wallet Launch to New Wallet Launch

See below a link to Mobile Wallet design. Still subject to feedback, so please let me know your thoughts.

I'll be making some small changes based on feedback from a call today with @jotto141 and @ottman - We'll meet again tomorrow at 10am ET / 14.00 GMT to discuss with @markeharding

WIP - https://invis.io/VRU68XT2NQU#/390940229_Wallet_-_Overview

closed

changed the description

opened

added 1 deleted label

changed finish date to Dec 25, 2019

changed finish date to Dec 24, 2019

mentioned in issue front#2137

changed the description

changed the description

Wallet Designs added to the main epic descriptor.

changed the description

@michaelfbradley This kind of dropdown is going to get super wide with certain selections. Is it ok to go with the "title on top" style we went with in analytics?

current	analytics

Good call @omadrid - I've made an update to the design where the label is brought outside, and positioned left of the input. This has been uploaded to InVision.

@michaelfbradley What should this link's destination look like?

Currently goes to this:

I'll have to add an extra page for this one. I'll work on this tomorrow, and notify you once completed @omadrid

Hey @omadrid - please see https://invis.io/KWU1X9IEB2U#/396448900_Wallet-_Tokens-_Overview-_On-Chain_Transfer for the On-Chain Transfer (Withdrawal) design

@michaelfbradley @markeharding I'm seeing "Ether" "ether" "ETH" and "Eth" in different locations to ostensibly refer to the same thing. It's already confusing enough without additional ambiguity... Can we make a decision on using just one or two instead of four? (e.g. Ether or ETH only, depending on context?).

Sincerely, the capitalisation police

Stick to Ether or ETH

I've updated designs accordingly.

added issue front#2177

@michaelfbradley Thanks for the quick responses :)

When you get to the 'Transfer to On-Chain' design will you please also take a look at the Metamask alternative links in the tokens > settings > custom section?

Right now they look like this:

create a new address	use your private key

Provide address -> https://invis.io/KWU1X9IEB2U#/396453407_Wallet-_Token-_Settings_-_Update_Address

Private key -> https://invis.io/KWU1X9IEB2U#/396454946_Wallet-_Token-_Settings_-_Download_Key

@michaelfbradley Regarding the form submission notifications -- are you thinking these should be incorporated into the existing notifications tool (the bell on the right of the topbar) or the more ephemeral form toaster that was used for Pro Settings? (Or something else?)

Toast notification. If you can simply use the current implementation that is being used by Pro Settings, that would be perfect.

changed the description

changed the description

added epic &110 as parent epic

mentioned in merge request front!680 (merged)

mentioned in merge request front!698

changed finish date to Jan 14, 2020

added issue minds/ux-design/ux#36

@omadrid @brianhatchet what is a real date we can trust for this epic to be completed?