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

Paper wallet dialog warning update #1337

Closed
wants to merge 1 commit into from

Conversation

peoplma
Copy link

@peoplma peoplma commented Mar 8, 2016

Make it clear in dialog that paper wallet privkeys are not saved.

Make it clear in dialog that paper wallet privkeys are not saved.
@langerhans
Copy link
Member

This warning only appears when you try to print paper wallets while being connected to any other nodes. Are you sure this is the right place for this warning? But maybe I'm putting too much faith in the users by assuming most of them would actually only generate paper wallets on an air gapped machine ;)

@peoplma
Copy link
Author

peoplma commented Mar 8, 2016

Yeah, people using it offline probably already know what they are doing. Hmm, maybe get rid of the if statement, change the dialog title to just "warning" and have it show up every time?

@langerhans
Copy link
Member

Pretty sure, the online warning should stay, I have hopes that people can learn ;)

Maybe we can put your warning somewhere else? Into the actual dialog? Or, we could check if the user actually clicked the "Print" button before they close the window?

@peoplma
Copy link
Author

peoplma commented Mar 8, 2016

Oh I wasn't suggesting getting rid of the online warning, just showing it no matter if you have connections or not (after all, 0 connections doesn't necessarily mean the computer is offline). So that the warning shows everytime you open the paper wallet generator.

But either of those other options would work too :)

@patricklodder patricklodder changed the base branch from 1.10.1-dev to 1.14.3-dev July 22, 2020 20:38
@patricklodder patricklodder added this to Needs work in 1.14.3 via automation Jul 22, 2020
@patricklodder patricklodder moved this from Needs work to To do in 1.14.3 Jul 22, 2020
@patricklodder
Copy link
Member

@langerhans do we still want to push this? Needs some rework / conflict resolution for 1.14.3.

@patricklodder patricklodder moved this from To do to Needs work in 1.14.3 Feb 3, 2021
@patricklodder
Copy link
Member

I'm taking this off the critical path for 1.14.3 as this still needs conflict resolution.

@patricklodder patricklodder removed this from Needs work in 1.14.3 Feb 22, 2021
@patricklodder patricklodder deleted the branch dogecoin:1.14.3-dev March 7, 2021 12:25
@patricklodder
Copy link
Member

If this still makes sense to change, please reopen against 1.14.4-dev or 1.21-dev.

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

Successfully merging this pull request may close these issues.

None yet

3 participants