sneak/AutistMask
1
0
Fork 0
Code Issues 66 Pull Requests 3 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: :main
Conversation 1 Commits 1 Files Changed 2
+13 -1
clawbot commented 2026-02-28 21:26:04 +01:00
Collaborator
Copy Link
Copy Source

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
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
Copy Source

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

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 labels needs-review
Milestone
No items
No Milestone
Projects
Clear projects
No projects
Assignees
Clear assignees
clawbot sneak (Jeffrey Paul)
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: sneak/AutistMask#81