Skip to content

docs(payments): document webhook-delivered payment events - #865

Merged
Jambox11 merged 1 commit into
mux-labs:mainfrom
Xaxxoo:fix/771-wire-payment-events-to-webhook
Sep 4, 2026
Merged

docs(payments): document webhook-delivered payment events#865
Jambox11 merged 1 commit into
mux-labs:mainfrom
Xaxxoo:fix/771-wire-payment-events-to-webhook

Conversation

@Xaxxoo

@Xaxxoo Xaxxoo commented Sep 1, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Add README documentation for payment domain events bridged to the webhook system
  • Documents the PaymentWebhookListener that bridges payment.created, payment.completed, and payment.failed events from the internal EventEmitter2 bus to WebhookEventEmitterService for outbound webhook delivery
  • The listener code and wiring already exist on main; this PR adds the missing documentation to satisfy the acceptance criteria

Closes #771

Test plan

  • Verify PaymentWebhookListener is registered in PaymentsModule providers
  • Verify listener handles all three payment events (created, completed, failed)
  • Confirm webhook dispatch errors are caught and logged without propagating
  • Run existing payment-webhook.listener.spec.ts tests

🤖 Generated with Claude Code

Add a section to the README documenting how internal payment domain
events (payment.created, payment.completed, payment.failed) are
bridged to the outbound webhook system via PaymentWebhookListener.

The listener and wiring code already exist; this commit adds the
missing documentation to satisfy the acceptance criteria.

Closes mux-labs#771

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@Jambox11
Jambox11 merged commit f1e5827 into mux-labs:main Sep 4, 2026
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