Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix(payment_methods): set requires-cvv to false for cards in customer payment methods list if making an off-session payment #4075

Merged
merged 1 commit into from
Mar 13, 2024

Conversation

vspecky
Copy link
Member

@vspecky vspecky commented Mar 13, 2024

Type of Change

  • Bugfix
  • New feature
  • Enhancement
  • Refactoring
  • Dependency updates
  • Documentation
  • CI/CD

Description

This PR updates the logic to decide the requires_cvv parameter in the customer payment methods list api to also depend on whether the payment is an off-session payment.

Additional Changes

  • This PR modifies the API contract
  • This PR modifies the database schema
  • This PR modifies application configuration/environment variables

Motivation and Context

For off-session payments, the cvv is not required, so it should not be mandatory for off-session payments.

How did you test it?

List Customer Payment Methods for normal payment (requires_cvv = true):
image

List Customer Payment Methods for off-session payment (requires_cvv = false):
image

Checklist

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

… payment methods list if making an off-session payment
@vspecky vspecky added A-core Area: Core flows C-bug Category: Bug S-waiting-on-review Status: This PR has been implemented and needs to be reviewed A-payment-methods Area: Payment Methods labels Mar 13, 2024
@vspecky vspecky added this to the March 2024 milestone Mar 13, 2024
@vspecky vspecky self-assigned this Mar 13, 2024
@vspecky vspecky requested a review from a team as a code owner March 13, 2024 11:47
@SanchithHegde SanchithHegde merged commit db25dac into main Mar 13, 2024
14 of 18 checks passed
@SanchithHegde SanchithHegde deleted the fix/off-session-requires-cvv branch March 13, 2024 12:13
@SanchithHegde SanchithHegde removed the S-waiting-on-review Status: This PR has been implemented and needs to be reviewed label Mar 13, 2024
pixincreate added a commit that referenced this pull request Mar 14, 2024
* 'main' of github.com:juspay/hyperswitch:
  chore(version): 2024.03.13.3
  fix(mandates): give higher precedence to connector mandate id over network txn id in mandates (#4073)
  ci(postman): fix bluesnap and checkout collections for saved card flow (#4071)
  fix: get valid test cards list based on wasm feature config (#4066)
  chore(version): 2024.03.13.2
  fix(payment_methods): set requires-cvv to false for cards in customer payment methods list if making an off-session payment (#4075)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-core Area: Core flows A-payment-methods Area: Payment Methods C-bug Category: Bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[BUG] requires_cvv being sent as true in customer payment methods list for off-session payments
3 participants