Skip to content

docs(point-of-sale): pin paymentvalidations to tender-confirm semantics - #4632

Merged
aaronschubert0 merged 1 commit into
2026-10-rcfrom
pos-payment-validations-tender-confirm-docs
Aug 18, 2026
Merged

docs(point-of-sale): pin paymentvalidations to tender-confirm semantics#4632
aaronschubert0 merged 1 commit into
2026-10-rcfrom
pos-payment-validations-tender-confirm-docs

Conversation

@aaronschubert0

Copy link
Copy Markdown
Contributor

What

Docs-only follow-up to #4611: pins PaymentValidationsEvent's dispatch timing, which was settled after that PR merged.

  • Fires when a tender is confirmed but not committed — after the amount is entered, before the payment is recorded (previously read "when staff selects a payment method", which was the pre-settlement wording).
  • One event per tender attempt; split payments dispatch one event per tender, each carrying its own amount.
  • Replaces the rejected two-event proposal (method-selected + post-tender).

Confirm-time is also why amount is per-tender meaningful: at selection time only the outstanding total exists; at confirm time the event carries the amount this tender actually charges.

References

@aaronschubert0
aaronschubert0 requested a review from vctrchu August 17, 2026 13:08

aaronschubert0 commented Aug 18, 2026

Copy link
Copy Markdown
Contributor Author

Merge activity

  • Aug 18, 9:47 AM UTC: A user started a stack merge that includes this pull request via Graphite.
  • Aug 18, 9:48 AM UTC: Graphite rebased this pull request as part of a merge.
  • Aug 18, 9:49 AM UTC: @aaronschubert0 merged this pull request with Graphite.

@aaronschubert0
aaronschubert0 force-pushed the pos-payment-validations-tender-confirm-docs branch from 09c0e98 to 3001369 Compare August 18, 2026 09:48
@aaronschubert0
aaronschubert0 merged commit 5d6090b into 2026-10-rc Aug 18, 2026
6 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