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 View on Explorer button #101

Merged
merged 3 commits into from
Mar 13, 2023
Merged

Add View on Explorer button #101

merged 3 commits into from
Mar 13, 2023

Conversation

JSKitty
Copy link
Member

@JSKitty JSKitty commented Mar 13, 2023

Abstract

This PR adds a simple button to the wallet UI to open the current 'master public key' in the user-preferred explorer.
Currently MPW offers no easy way of viewing the xpub, this is a quick workaround while also allowing an easy shortcut to the explorer.

image

Additionally this PR fixes a small bug in which a non-HD wallet in view-only mode would fail to hide the "Refresh Address" button, this is only intended for HD wallets.

What does this PR address?

It assists with the inability to view the xpub as well as granting an explorer shortcut.

What features or improvements were added?

A Dashboard shortcut to the explorer page of the current wallet public key.

How does this benefit users?

Users can access their xpub easier or simply quick-access an integrated PIVX explorer, alongside monitoring their address for transactions while away/outside from MPW.

Alternatively, users could find it useful to import their xpub in to another MPW instance, purely for View-Only monitoring purposes.

@JSKitty JSKitty added the Enhancement New feature or request label Mar 13, 2023
@JSKitty JSKitty self-assigned this Mar 13, 2023
Copy link
Member

@Duddino Duddino left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

tACK everything works as expected

@JSKitty JSKitty merged commit 92d7568 into master Mar 13, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Enhancement New feature or request
Projects
Development

Successfully merging this pull request may close these issues.

3 participants