-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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(router): add support for Samsung Pay payment method #5955
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
ShankarSinghC
force-pushed
the
samsung_pay
branch
from
September 19, 2024 08:15
48a2535
to
74dfba4
Compare
hyperswitch-bot
bot
added
the
M-api-contract-changes
Metadata: This PR involves API contract changes
label
Sep 19, 2024
ShankarSinghC
changed the title
feat(router): apple support for Samsung Pay payment method
feat(router): add support for Samsung Pay payment method
Sep 19, 2024
ShankarSinghC
force-pushed
the
samsung_pay
branch
from
September 19, 2024 13:24
b580310
to
19ba83b
Compare
ShankarSinghC
force-pushed
the
samsung_pay
branch
2 times, most recently
from
September 19, 2024 14:44
e401e06
to
d1b3080
Compare
ShankarSinghC
added
C-feature
Category: Feature request or enhancement
A-payment-methods
Area: Payment Methods
labels
Sep 19, 2024
ShankarSinghC
force-pushed
the
samsung_pay
branch
from
September 19, 2024 17:09
d1b3080
to
e71f3a6
Compare
SanchithHegde
previously approved these changes
Sep 19, 2024
deepanshu-iiitu
previously approved these changes
Sep 20, 2024
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Approving on behalf of connectors
Chethan-rao
previously approved these changes
Sep 20, 2024
ShankarSinghC
dismissed stale reviews from Chethan-rao, deepanshu-iiitu, and SanchithHegde
via
September 20, 2024 07:28
5de78ac
ShankarSinghC
requested review from
Chethan-rao,
SanchithHegde and
deepanshu-iiitu
September 20, 2024 07:30
apoorvdixit88
approved these changes
Sep 20, 2024
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Dashboard specific change looks fine
Chethan-rao
approved these changes
Sep 20, 2024
SanchithHegde
approved these changes
Sep 20, 2024
deepanshu-iiitu
approved these changes
Sep 20, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
A-payment-methods
Area: Payment Methods
C-feature
Category: Feature request or enhancement
M-api-contract-changes
Metadata: This PR involves API contract changes
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Type of Change
Description
This pr is to support for Samsung pay as a payment method.
This pr also adds a new field in merchant connector account create
connector_wallet_details
which is required to collect the required credentials for Samsung Pay. As this column in the data base stores the data in the encrypted format we also want to migrate apple pay certificates from metadata toconnector_wallet_details
, for which we are inserting apple pay details passed in the metadata to connector_wallet_details.Once the frontend changes are done to pass the apple pay credentials in the connector_wallet_details this fallback can be removed.
Additional Changes
Motivation and Context
How did you test it?
Test Samsung Pay flow
-> Enable Samsung Pay payment method for the connector
-> Create a payment
-> Session call
-> Confirm call
The status will be failed as Samsung does not have a test card and cybersource returns invalid data error for live cards
Checklist
cargo +nightly fmt --all
cargo clippy