-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
Improve xmr proof service #6217
Improve xmr proof service #6217
Conversation
Add check for duplicate txHash Rename wrongly names variables at startRequestsIfValid
@jmacxx Could you please review this PR? Thanks! |
@ripcurlx yes, I am in process of reviewing/testing the code. Will let you know ASAP. This popup text could probably use some revision from yourself / @m52go / @w0000000t to make it more understandable. It is shown at startup (only users with XMR accounts). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ACK
I tested mainly the subaddress generation - works fine for me.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
utACK - based on #6217 (review)
I thought I had replied already, sorry for the delay! I probably left the browser tab hanging and it went lost.
|
@w0000000t thanks that looks good to me. How about this revision:
|
The amount of text increased, but likewise for information density, I am cool with it |
I am currently testing this, and it didn't show on startup for me, or at least I cannot be 100% sure it didn't show when starting 1.9.3 testing instance with existing data directory with existing XMR accounts as I did close the popups quite shortly thereafter 😅 |
The popup moved to |
Fixes some issues brought up by kayabanerve