hyperswitch icon indicating copy to clipboard operation
hyperswitch copied to clipboard

feat(core): added multiple payment_attempt support for payment_intent

Open Abhicodes-crypto opened this issue 2 years ago • 2 comments

Type of Change

  • [ ] Bugfix
  • [x] New feature
  • [ ] Enhancement
  • [ ] Refactoring
  • [ ] Dependency updates

Description

A column 'attempt_id' was added to payment_intent table while ensuring that the current kv or postgres flows don't get disturbed. Having multiple payment_attempts associated with a single payment_intent helps us implement 'Retries'.

Additional Changes

  • [x] This PR modifies the database schema
  • [ ] This PR modifies application configuration/environment variables

Motivation and Context

How did you test it?

Manual, verified if the payment flows were working as before. KV image image

POSTGRES image image

Checklist

  • [x] I formatted the code cargo +nightly fmt --all
  • [x] I addressed lints thrown by cargo clippy
  • [x] I reviewed submitted code
  • [ ] I added unit tests for my changes where possible
  • [ ] I added a CHANGELOG entry if applicable

Abhicodes-crypto avatar Jan 20 '23 12:01 Abhicodes-crypto

Will Reopen the PR when I'm done with all the changes.

Abhicodes-crypto avatar Feb 14 '23 05:02 Abhicodes-crypto

Will Reopen the PR when I'm done with all the changes.

Converting this to a draft PR instead.

SanchithHegde avatar Feb 14 '23 06:02 SanchithHegde