-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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(connector): [Bluesnap] Remove wallet call #1620
Conversation
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.
@Sangamesh26 Please attach suitable labels
@@ -504,84 +504,6 @@ impl api::PaymentSession for Bluesnap {} | |||
impl ConnectorIntegration<api::Session, types::PaymentsSessionData, types::PaymentsResponseData> | |||
for Bluesnap | |||
{ | |||
fn get_headers( |
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.
@Sangamesh26 Do not remove this code, let it be and add comments. Revert only the core changes
@@ -299,96 +286,6 @@ impl From<api_models::payments::ApplepayPaymentMethod> for ApplepayPaymentMethod | |||
} | |||
} | |||
|
|||
impl TryFrom<&types::PaymentsSessionRouterData> for BluesnapCreateWalletToken { |
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.
Same here.
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.
Let's not remove the code for session call
@Sangamesh26 Please add necessary labels to the PR |
Co-authored-by: Sahkal Poddar <sahkal.poddar@juspay.in> Co-authored-by: Pa1NarK <69745008+pixincreate@users.noreply.github.com>
Co-authored-by: Sahkal Poddar <sahkal.poddar@juspay.in> Co-authored-by: Pa1NarK <69745008+pixincreate@users.noreply.github.com>
Type of Change
Description
Remove Wallet call form bluesnap which was earlier treated a session call.
We get session object now from apple only instead of bluesnap.
Motivation and Context
Get session response from Apple pay instead of getting from connector
How did you test it?
Manual
Checklist
cargo +nightly fmt --all
cargo clippy