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

feat: support gpay and applepay session response for all connectors #839

Merged
merged 15 commits into from
Apr 20, 2023

Conversation

Sangamesh26
Copy link
Contributor

@Sangamesh26 Sangamesh26 commented Apr 5, 2023

Type of Change

  • New feature

Description

Earlier session token for gpay was only supported for Adyen and applepay for Applepay as connector.So, now gpay and applepay session token is supported by all the connectors which enable them as payment_methods.

Removed Applepay as a connector and get the response from applepay server in the session_flow itself. Ideally Applepay should not be treated as a connector, it was earlier hided in dashboard by some filters in the dashboard side.

Motivation and Context

Support applepay and gpay for all connectors which enable them as payment_methods for session_token.

How did you test it?

Manual

Checklist

  • I formatted the code cargo +nightly fmt --all
  • I addressed lints thrown by cargo clippy
  • I reviewed submitted code

@Sangamesh26 Sangamesh26 requested review from a team as code owners April 5, 2023 10:22
@Sangamesh26 Sangamesh26 self-assigned this Apr 5, 2023
@Sangamesh26 Sangamesh26 added A-core Area: Core flows C-feature Category: Feature request or enhancement S-waiting-on-review Status: This PR has been implemented and needs to be reviewed labels Apr 5, 2023
@jarnura jarnura added the S-waiting-on-author Status: This PR is incomplete or needs to address review comments label Apr 5, 2023
@Sangamesh26 Sangamesh26 removed the S-waiting-on-author Status: This PR is incomplete or needs to address review comments label Apr 6, 2023
@Sangamesh26 Sangamesh26 requested a review from jarnura April 6, 2023 07:11
jarnura
jarnura previously approved these changes Apr 6, 2023
@jarnura jarnura added this to the April'23 release milestone Apr 14, 2023
Narayanbhat166
Narayanbhat166 previously approved these changes Apr 18, 2023
@jarnura jarnura added S-waiting-on-author Status: This PR is incomplete or needs to address review comments and removed S-waiting-on-review Status: This PR has been implemented and needs to be reviewed labels Apr 19, 2023
@Sangamesh26 Sangamesh26 added S-waiting-on-review Status: This PR has been implemented and needs to be reviewed and removed S-waiting-on-author Status: This PR is incomplete or needs to address review comments labels Apr 19, 2023
@jarnura jarnura added S-ready-for-merge and removed S-waiting-on-review Status: This PR has been implemented and needs to be reviewed labels Apr 20, 2023
@jarnura jarnura added this pull request to the merge queue Apr 20, 2023
Merged via the queue into main with commit d23e14c Apr 20, 2023
@SanchithHegde SanchithHegde deleted the applepay_all_connectors branch April 21, 2023 04:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-core Area: Core flows C-feature Category: Feature request or enhancement
Projects
No open projects
Status: Merged
Development

Successfully merging this pull request may close these issues.

4 participants