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

History: fix payment proof button #3910

Merged
merged 1 commit into from
Jun 9, 2022

Conversation

reemuru
Copy link
Contributor

@reemuru reemuru commented May 5, 2022

Closes #3909

Adding support for reserve proof added an additional
argument to the getProofClicked signal. The payment
proof button on the transaction list from history.qml
does not pass this argument and fails with insufficient
argument error.

Fixes: 0f67580 ("Advanced: ReserveProof: Add support for reserve proof")

Temporary fix: Use Advanced tab on the GUI or use CLI to generate tx proof (OutProof)
Edit: tested locally using docker reproducible build against master branch

Adding support for reserve proof added an additional
argument to the getProofClicked signal. The payment
proof button on the transaction list from history.qml
does not pass this argument and fails with insufficient
argument error.

Fixes: 0f67580 ("Advanced: ReserveProof: Add support for reserve proof")
@reemuru
Copy link
Contributor Author

reemuru commented May 5, 2022

Steps to test

  1. Download the latest GUI release
  2. Send a transaction
  3. Go to transactions sub menu
  4. Click on a transaction -> 'Generate Payment Proof' icon
  5. No response
  6. Build this pull request and repeat 3-4
  7. Payment proof (tx proof) should display

proof-3909

tx-proof-v17 3 2

@reemuru reemuru marked this pull request as ready for review May 5, 2022 18:06
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.

ty :)

@luigi1111 luigi1111 merged commit a67865b into monero-project:master Jun 9, 2022
@reemuru reemuru mentioned this pull request Jun 23, 2022
@reemuru reemuru deleted the bug-3909 branch June 23, 2022 14:39
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.

Bug v0.17.3.2 Generate payment proof button
3 participants