Skip to content
New issue

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

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

Already on GitHub? # to your account

fix(router): update nick_name only if card_token.card_holder_name is non empty and populate additional card_details from payment_attempt if not present in the locker #6320

Merged

Conversation

sai-harsha-vardhan
Copy link
Contributor

Type of Change

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

Description

Main PR - #6308

Additional Changes

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

Motivation and Context

How did you test it?

Test cases in main PR

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

… populate additional card_details from payment_attempt if not present in the locker
@sai-harsha-vardhan sai-harsha-vardhan added A-core Area: Core flows C-bug Category: Bug labels Oct 15, 2024
@sai-harsha-vardhan sai-harsha-vardhan self-assigned this Oct 15, 2024
@sai-harsha-vardhan sai-harsha-vardhan requested review from a team as code owners October 15, 2024 10:14
Copy link

semanticdiff-com bot commented Oct 15, 2024

Review changes with SemanticDiff.

Analyzed 2 of 2 files.

Overall, the semantic diff is 14% smaller than the GitHub diff.

Filename Status
✔️ crates/router/src/core/payment_methods.rs Analyzed
✔️ crates/router/src/core/payments/helpers.rs 14.41% smaller

@sai-harsha-vardhan sai-harsha-vardhan changed the base branch from main to hotfix-2024.10.04.0 October 15, 2024 10:16
@sai-harsha-vardhan sai-harsha-vardhan removed request for a team October 15, 2024 10:16
@sai-harsha-vardhan sai-harsha-vardhan added the S-waiting-on-review Status: This PR has been implemented and needs to be reviewed label Oct 15, 2024
@likhinbopanna likhinbopanna merged commit 4fdf5a5 into hotfix-2024.10.04.0 Oct 15, 2024
18 checks passed
@likhinbopanna likhinbopanna deleted the fix-cardholdername-temp-card-flow-hotfix branch October 15, 2024 11:40
@SanchithHegde SanchithHegde removed the S-waiting-on-review Status: This PR has been implemented and needs to be reviewed label Oct 27, 2024
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
A-core Area: Core flows C-bug Category: Bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants