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

Add dialog for displaying seed phrase to JM-Qt #418

Merged
merged 1 commit into from
Oct 25, 2019

Conversation

chris-belcher
Copy link
Contributor

@chris-belcher chris-belcher commented Oct 17, 2019

I noticed the Joinmarket-Qt application didn't have a way to display the seed phrase of a loaded wallet. This seemed like an oversight.

Also I made the display words dialog use bold font for the words and extension.
Also I rearranged some of the QAction lines so they match the order that the options appear in the GUI.

I've tested with and without an extension.

@AdamISZ
Copy link
Member

AdamISZ commented Oct 17, 2019

Oh wow, I'd totally forgot we didn't have that. Thanks!

@chris-belcher
Copy link
Contributor Author

The travis build seemed to have failed because it stalled, not anything to do with the PR code.

scripts/joinmarket-qt.py Outdated Show resolved Hide resolved
@chris-belcher chris-belcher force-pushed the create-show-seed-dialog branch from 008bb89 to 815b8a3 Compare October 17, 2019 19:03
@chris-belcher chris-belcher force-pushed the create-show-seed-dialog branch from 815b8a3 to 9dee0d7 Compare October 25, 2019 19:30
@chris-belcher
Copy link
Contributor Author

Fixed after #359, tested myself.

chris-belcher added a commit that referenced this pull request Oct 25, 2019
9dee0d7 Add dialog for displaying seed phrase to JM-Qt (chris-belcher)

Tree-SHA512: fa8b30c8078f85e9c877d23cff426fe031b74c2adff22609b59c0642ebfb7779986fbb7142906a707a4d1458db08823691d226ee80ca3154cc2cb04da89a86fb
@chris-belcher chris-belcher merged commit 9dee0d7 into master Oct 25, 2019
@chris-belcher chris-belcher deleted the create-show-seed-dialog branch October 25, 2019 19:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants