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

Advanced: ReserveProof: Add support for reserve proof #3828

Merged

Conversation

reemuru
Copy link
Contributor

@reemuru reemuru commented Jan 21, 2022

Closes #3826

This change adds the ability to prove and check a reserve proof
to this existing "Prove/check" advanced menu. If the amount line has
a valid amount less than that of the current account index this input
will drive the logic for generating a reserve proof. Checking a reserve
proof is accomplished by validating the address and verifying that
signature.indexOf("ReserveProofV") === 0. The result displays the total
and spent amount of the proof.

reserve_proof_layout

reserve_proof_test

check_reserve_proof_layout

check_reserve_proof

This change adds the ability to prove and check a reserve proof
to this existing "Prove/check" advanced menu. If the amount line has
a valid amount less than that of the current account index this input
will drive the logic for generating a reserve proof. Checking a reserve
proof is accomplished by validating the address and verifying that
signature.indexOf("ReserveProofV") === 0. The result displays the total
and spent amount of the proof.
Copy link
Collaborator

@selsta selsta left a comment

Choose a reason for hiding this comment

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

Looks good, thank you. Left two minor comments.

pages/TxKey.qml Outdated Show resolved Hide resolved
pages/TxKey.qml Outdated Show resolved Hide resolved
Fixes: 0f67580
This commit updates qsTr() to avoid breaking previous
translations. Also removes an erroneous whitespace.
@luigi1111 luigi1111 merged commit 1610b93 into monero-project:master Mar 3, 2022
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.

[Feature Request] Implement Reserve Proofs
3 participants