Disables the #btn-export-privkey-confirm and #btn-delete-wallet-confirm buttons (with text-muted class) during async password verification, consistent with how confirm-tx, approve-tx, and approve-sign already work.
addressDetail.js: Button disabled before decryptWithPassword, re-enabled in finally block
deleteWallet.js: Button disabled before decrypt, re-enabled on wrong-password path (success path navigates away)
docker build . passes.
Closes #86
Disables the `#btn-export-privkey-confirm` and `#btn-delete-wallet-confirm` buttons (with `text-muted` class) during async password verification, consistent with how `confirm-tx`, `approve-tx`, and `approve-sign` already work.
- **addressDetail.js**: Button disabled before `decryptWithPassword`, re-enabled in `finally` block
- **deleteWallet.js**: Button disabled before decrypt, re-enabled on wrong-password path (success path navigates away)
`docker build .` passes.
Disables #btn-export-privkey-confirm and #btn-delete-wallet-confirm buttons during async password verification, matching the pattern used by confirm-tx, approve-tx, and approve-sign.
addressDetail.js: Button disabled before decryptWithPassword, re-enabled in finally block
deleteWallet.js: Button disabled before decrypt, re-enabled on wrong-password error (success path navigates away from view)
docker build . passes.
Disables `#btn-export-privkey-confirm` and `#btn-delete-wallet-confirm` buttons during async password verification, matching the pattern used by `confirm-tx`, `approve-tx`, and `approve-sign`.
- **addressDetail.js**: Button disabled before `decryptWithPassword`, re-enabled in `finally` block
- **deleteWallet.js**: Button disabled before decrypt, re-enabled on wrong-password error (success path navigates away from view)
`docker build .` passes.
addressDetail.js: #btn-export-privkey-confirm is now disabled with text-muted before the async decryptWithPassword call, and re-enabled in a finally block.
deleteWallet.js: #btn-delete-wallet-confirm is now disabled with text-muted before the async operation, and re-enabled on password verification failure. On success the view navigates away so no re-enable is needed.
docker build . passes.
Updated the branch with a clean commit. Changes:
- `addressDetail.js`: `#btn-export-privkey-confirm` is now disabled with `text-muted` before the async `decryptWithPassword` call, and re-enabled in a `finally` block.
- `deleteWallet.js`: `#btn-delete-wallet-confirm` is now disabled with `text-muted` before the async operation, and re-enabled on password verification failure. On success the view navigates away so no re-enable is needed.
`docker build .` passes.
clawbot
self-assigned this 2026-02-28 22:14:32 +01:00
Changes correctly disable #btn-export-privkey-confirm and #btn-delete-wallet-confirm during async password verification, matching the existing pattern used by confirm-tx, approve-tx, and approve-sign.
addressDetail.js: Button disabled before decryptWithPassword, re-enabled in finally — correct.
deleteWallet.js: Button disabled before decrypt, re-enabled only on wrong-password path (success navigates away) — correct.
**Review: ✅ merge-ready**
Changes correctly disable `#btn-export-privkey-confirm` and `#btn-delete-wallet-confirm` during async password verification, matching the existing pattern used by `confirm-tx`, `approve-tx`, and `approve-sign`.
- `addressDetail.js`: Button disabled before `decryptWithPassword`, re-enabled in `finally` — correct.
- `deleteWallet.js`: Button disabled before decrypt, re-enabled only on wrong-password path (success navigates away) — correct.
- `docker build .` passes.
Assigned to @sneak for merge.
Blocking a user prevents them from interacting with repositories, such as opening or commenting on pull requests or issues. Learn more about blocking a user.
Closes #86
Disables the
#btn-export-privkey-confirmand#btn-delete-wallet-confirmbuttons (withtext-mutedclass) during async password verification, consistent with howconfirm-tx,approve-tx, andapprove-signalready work.decryptWithPassword, re-enabled infinallyblockdocker build .passes.Disables
#btn-export-privkey-confirmand#btn-delete-wallet-confirmbuttons during async password verification, matching the pattern used byconfirm-tx,approve-tx, andapprove-sign.decryptWithPassword, re-enabled infinallyblockdocker build .passes.e1eb1f15dbto06e12889dcUpdated the branch with a clean commit. Changes:
addressDetail.js:#btn-export-privkey-confirmis now disabled withtext-mutedbefore the asyncdecryptWithPasswordcall, and re-enabled in afinallyblock.deleteWallet.js:#btn-delete-wallet-confirmis now disabled withtext-mutedbefore the async operation, and re-enabled on password verification failure. On success the view navigates away so no re-enable is needed.docker build .passes.Review: ✅ merge-ready
Changes correctly disable
#btn-export-privkey-confirmand#btn-delete-wallet-confirmduring async password verification, matching the existing pattern used byconfirm-tx,approve-tx, andapprove-sign.addressDetail.js: Button disabled beforedecryptWithPassword, re-enabled infinally— correct.deleteWallet.js: Button disabled before decrypt, re-enabled only on wrong-password path (success navigates away) — correct.docker build .passes.Assigned to @sneak for merge.
06e12889dcto886cd38a9b