Skip to content

feat(popup): add payment request status toast#179

Merged
Kelbie merged 1 commit intomainfrom
feat/popup-payment-request-status
Mar 6, 2026
Merged

feat(popup): add payment request status toast#179
Kelbie merged 1 commit intomainfrom
feat/popup-payment-request-status

Conversation

@Kelbie
Copy link
Contributor

@Kelbie Kelbie commented Mar 6, 2026

What / Why

  • add a dedicated payment-request variant to the shared payment status popup flow
  • keep payment-request send feedback declarative instead of using an ad-hoc Nostr success popup
  • let claim confirmation come from coco send:finalized events so the toast stays aligned with the actual send lifecycle

What changed

  • extended the runtime payment status store and popup/toast config to support payment-request
  • updated payment-request send flows to open the shared status toast with Waiting for recipient copy
  • changed the payment status listener to confirm an active payment-request toast in place when the recipient claims the send
  • kept the transaction deep-link behavior for confirmed states

Test plan

  1. Run npm run lint
  2. Run npm run type-check
  3. Run npm run knip
  4. Run npm run pretty:check and note it still fails because of the unrelated existing uniwind-types.d.ts worktree change that is not included in this commit

Notes / Risks

  • Manual in-app verification is still needed for the full payment-request flow
  • Unrelated worktree changes remain in uniwind-types.d.ts and coco, and are intentionally excluded from this PR

Made with Cursor

Route payment-request sends through the shared payment status popup so the flow stays declarative and claim confirmation comes from coco send lifecycle events.

Made-with: Cursor
@Kelbie Kelbie merged commit 4215455 into main Mar 6, 2026
1 check 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.

1 participant