Skip to content

feat(gui): show recovery button on page#1084

Merged
binarybaron merged 1 commit into
masterfrom
feat/gui-recovery-button-on-page
Jun 8, 2026
Merged

feat(gui): show recovery button on page#1084
binarybaron merged 1 commit into
masterfrom
feat/gui-recovery-button-on-page

Conversation

@binarybaron

Copy link
Copy Markdown

No description provided.

@binarybaron binarybaron force-pushed the feat/gui-recovery-button-on-page branch from 76153c0 to dcd7e59 Compare June 7, 2026 05:17
@binarybaron

Copy link
Copy Markdown
Author

bugbot run

@cursor cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit dcd7e59. Configure here.

>
Redeem manually
</SwapMoneroRecoveryButton>
)}

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Duplicate recovery modals open

Medium Severity

Each SwapMoneroRecoveryButton mounts its own MoneroRecoveryKeysDialog, which opens when shared moneroRecovery state matches the swap. With the new in-progress buttons plus the existing alert button, one successful fetch can render two open dialogs for the same keys.

Additional Locations (1)
Fix in Cursor Fix in Web

Reviewed by Cursor Bugbot for commit dcd7e59. Configure here.

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

@binarybaron Is this a problem?

@binarybaron binarybaron merged commit c057d5f into master Jun 8, 2026
73 of 79 checks passed
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.

2 participants