Skip to content

Conversation

@gopikrishna000
Copy link
Contributor

@gopikrishna000 gopikrishna000 commented Dec 5, 2025

Description

Refund and Rsync flow of paypal implementation

Motivation and Context

Refund and Rsync flow of paypal implementation

Additional Changes

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

How did you test it?

https://github.com/juspay/hyperswitch-cloud/issues/12818
https://github.com/juspay/hyperswitch-cloud/issues/12819

@gopikrishna000 gopikrishna000 requested a review from a team as a code owner December 5, 2025 09:58
}
PaymentMethodData::MandatePayment
| PaymentMethodData::Reward
PaymentMethodData::MandatePayment => {
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

MandatePayment should be handled in RepeatPayment flow

macros::macro_connector_implementation!(
connector_default_implementations: [get_content_type, get_error_response_v2],
connector: Paypal,
curl_request: Json(paypal::PaypalRefundRequest),
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
curl_request: Json(paypal::PaypalRefundRequest),
curl_request: Json(PaypalRefundRequest),

macros::macro_connector_implementation!(
connector_default_implementations: [get_content_type, get_error_response_v2],
connector: Paypal,
curl_response: paypal::RefundSyncResponse,
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
curl_response: paypal::RefundSyncResponse,
curl_response: RefundSyncResponse,

@AmitsinghTanwar007 AmitsinghTanwar007 added this pull request to the merge queue Dec 8, 2025
Merged via the queue into main with commit 70cccb9 Dec 8, 2025
14 of 15 checks passed
@AmitsinghTanwar007 AmitsinghTanwar007 deleted the feat/paypal-refund branch December 8, 2025 07:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants