Skip to content
This repository has been archived by the owner on Dec 11, 2019. It is now read-only.

Add alert dialog for wallet backup invitation #4847

Closed
bradleyrichter opened this issue Oct 16, 2016 · 13 comments
Closed

Add alert dialog for wallet backup invitation #4847

bradleyrichter opened this issue Oct 16, 2016 · 13 comments
Assignees
Labels
design A design change, especially one which needs input from the design team. feature/rewards needs-info Another team member needs information from the PR/issue opener.

Comments

@bradleyrichter
Copy link
Contributor

bradleyrichter commented Oct 16, 2016

When a user first adds funds to their auto-created Brave wallet, they should be invited to Backup their wallet. The dialog should open as a modal alert after they have completed the initial add-funds step.

image

cc @mrose17 @bbondy

@bradleyrichter bradleyrichter added this to the 0.12.6dev milestone Oct 16, 2016
@mrose17 mrose17 modified the milestones: 0.12.7dev, 0.12.6dev Oct 17, 2016
@mrose17 mrose17 modified the milestones: 0.12.8dev, 0.12.7dev Oct 22, 2016
@brave-office brave-office modified the milestones: 1.1.0, 0.12.8dev Oct 25, 2016
@brave-office brave-office modified the milestones: 0.13.2, 1.1.0 Jan 10, 2017
@jkup jkup assigned jkup and unassigned diracdeltas Jan 11, 2017
@mrose17 mrose17 modified the milestones: 0.13.5, 0.13.4 Feb 14, 2017
@bsclifton
Copy link
Member

Moving to 0.13.7

@bsclifton bsclifton modified the milestones: 0.13.7, 0.13.6 Mar 13, 2017
@NejcZdovc NejcZdovc self-assigned this Mar 17, 2017
@NejcZdovc
Copy link
Contributor

@bradleyrichter should we use our regular alert or create a custom one?

@bradleyrichter
Copy link
Contributor Author

bradleyrichter commented Mar 25, 2017 via email

@NejcZdovc
Copy link
Contributor

@bradleyrichter in mock user can only see backup now or remind me later, what if he doesn't want to see it at all? If we keep remind me later, when should we trigger it again? After another deposit?

@bradleyrichter
Copy link
Contributor Author

bradleyrichter commented Mar 27, 2017 via email

NejcZdovc added a commit to NejcZdovc/browser-laptop that referenced this issue Apr 7, 2017
@NejcZdovc NejcZdovc modified the milestones: 0.14.3, 0.14.2 Apr 9, 2017
@NejcZdovc
Copy link
Contributor

blocked by #8137

@luixxiul
Copy link
Contributor

luixxiul commented Jun 4, 2017

#8137 was merged.

@ghost ghost removed this from the 1.0.0 milestone Sep 26, 2017
@ghost ghost added the design A design change, especially one which needs input from the design team. label Sep 26, 2017
@bsclifton
Copy link
Member

@bsclifton bsclifton added this to the Triage Backlog milestone Nov 27, 2017
@NejcZdovc
Copy link
Contributor

@mrose17 do we have any mechanism to detect when first contribution is made?

@NejcZdovc NejcZdovc added the needs-info Another team member needs information from the PR/issue opener. label Feb 19, 2018
@mrose17
Copy link
Member

mrose17 commented Feb 20, 2018

@NejcZdovc - you'd have to look at the history for the earliest entry

       transactions[...].submissionStamp

@NejcZdovc
Copy link
Contributor

blocked on #13504. This PR will introduce backup flag that we can then use in this PR

@NejcZdovc NejcZdovc added PR/blocked and removed needs-info Another team member needs information from the PR/issue opener. labels Apr 9, 2018
@NejcZdovc
Copy link
Contributor

@bradleyrichter do we still want to add this now that we added #13504

@NejcZdovc NejcZdovc added needs-info Another team member needs information from the PR/issue opener. and removed PR/blocked labels Apr 26, 2018
@NejcZdovc
Copy link
Contributor

closing per discussion with @bradleyrichter

@NejcZdovc NejcZdovc removed this from the Triage Backlog milestone Apr 26, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
design A design change, especially one which needs input from the design team. feature/rewards needs-info Another team member needs information from the PR/issue opener.
Projects
None yet
Development

No branches or pull requests

9 participants