Skip to content

Conversation

@deveasyclick
Copy link
Owner

@deveasyclick deveasyclick commented Nov 23, 2025

This PR adds the initial reconciliation wiring for the payments flow. It prepares the payment module to receive and persist provider responses, and introduces a consumer used to process async reconciliation jobs. The primary goal is to enable reliable reconciliation between our internal payment attempts and external provider outcomes (InterSwitch, VTPass).

What changed

  • PaymentModule

    • Registered InterSwitchModule and VTPassModule so provider-specific integrations are available to the payment flow.
    • Added PaymentConsumer to providers — this consumer will process reconciliation jobs (e.g., poll provider status, handle callbacks, or process queue messages that reconcile attempts).
  • PaymentRepository

    • updatePaymentAttempt now accepts and stores responsePayload (instead of requestPayload) so raw provider responses are persisted. This is required to:

      • Compare provider responses to the original attempt (reconciliation).
      • Log provider data for audits and debugging.
      • Drive state transitions (success / failed / pending) during reconciliation.

Yusuf added 2 commits November 23, 2025 22:59
…sumer

- Added InterSwitchModule and VTPassModule to PaymentModule imports
- Added PaymentConsumer to providers
- Prepared module for external provider processing and background workflow
- Updated PaymentRepository.updatePaymentAttempt signature
- Removed requestPayload from update path
- Ensures provider response data is logged accurately
@vercel
Copy link

vercel bot commented Nov 23, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

1 Skipped Deployment
Project Deployment Preview Comments Updated (UTC)
billpay Skipped Skipped Nov 23, 2025 10:31pm

@deveasyclick deveasyclick changed the title Feat/payment add integrations and response payload Add payment reconciliation logic Nov 23, 2025
@deveasyclick deveasyclick merged commit 2374a73 into main Nov 23, 2025
3 of 4 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.

1 participant