feat(router): add support for Samsung Pay payment method#5955
Merged
likhinbopanna merged 7 commits intomainfrom Sep 20, 2024
Merged
feat(router): add support for Samsung Pay payment method#5955likhinbopanna merged 7 commits intomainfrom
likhinbopanna merged 7 commits intomainfrom
Conversation
48a2535 to
74dfba4
Compare
b580310 to
19ba83b
Compare
e401e06 to
d1b3080
Compare
d1b3080 to
e71f3a6
Compare
SanchithHegde
previously approved these changes
Sep 19, 2024
deepanshu-iiitu
previously approved these changes
Sep 20, 2024
Contributor
deepanshu-iiitu
left a comment
There was a problem hiding this comment.
Approving on behalf of connectors
Chethan-rao
previously approved these changes
Sep 20, 2024
5de78ac
apoorvdixit88
approved these changes
Sep 20, 2024
Contributor
apoorvdixit88
left a comment
There was a problem hiding this comment.
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
This file contains hidden or 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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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_detailswhich 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 --allcargo clippy