sneak/AutistMask
1
0
Fork 0
Code Issues 1 Pull Requests 1 Actions Packages Projects Releases Wiki Activity

fix: persist confirm-tx view across popup close/reopen #81

Closed
clawbot wants to merge 1 commits from fix/77-confirm-tx-persist into main
pull from: fix/77-confirm-tx-persist
merge into: sneak:main
Conversation 1 Commits 1 Files Changed 2 +13 -1
clawbot commented 2026-02-28 21:26:04 +01:00
Collaborator
Copy Link

The confirm-tx view was not in RESTORABLE_VIEWS, so closing and reopening the popup would fall back to the main view, losing the pending transaction confirmation.

Changes:

  • Save pendingTx data to state.viewData when showing confirm-tx
  • Add confirm-tx to RESTORABLE_VIEWS
  • Add restore logic in restoreView() to re-render confirm-tx from persisted data

docker build . passes.

Closes #77

The `confirm-tx` view was not in `RESTORABLE_VIEWS`, so closing and reopening the popup would fall back to the main view, losing the pending transaction confirmation. Changes: - Save `pendingTx` data to `state.viewData` when showing confirm-tx - Add `confirm-tx` to `RESTORABLE_VIEWS` - Add restore logic in `restoreView()` to re-render confirm-tx from persisted data `docker build .` passes. Closes #77
clawbot added 1 commit 2026-02-28 21:26:04 +01:00
fix: persist confirm-tx view across popup close/reopen
All checks were successful
check / check (push) Successful in 22s
Details
d2f7284975 
The confirm-tx view was not in RESTORABLE_VIEWS, so closing and
reopening the popup would fall back to the main view, losing the
pending transaction confirmation.

Save pendingTx data to state.viewData when showing confirm-tx and
add confirm-tx to the set of restorable views with proper restore
logic.

Closes #77
clawbot added the needs-review label 2026-02-28 21:26:16 +01:00
clawbot closed this pull request 2026-02-28 21:28:15 +01:00
clawbot commented 2026-02-28 21:28:16 +01:00
Author
Collaborator
Copy Link

Closing as duplicate of #79.

Closing as duplicate of #79.
clawbot self-assigned this 2026-02-28 21:37:59 +01:00
clawbot removed their assignment 2026-02-28 21:38:11 +01:00
All checks were successful
check / check (push) Successful in 22s
Required
Details

Pull request closed

Please reopen this pull request to perform a merge.
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
Clear labels
merge-ready
needs-checks
needs-rebase
needs-review
needs-rework
No Label needs-review
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
clawbot sneak
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: sneak/AutistMask#81
Delete Branch "fix/77-confirm-tx-persist"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?