-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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): [Payme] Add Void flow to Payme #3817
Conversation
}) | ||
} else { | ||
Ok(types::PaymentsResponseData::TransactionResponse { | ||
resource_id: types::ResponseId::NoResponseId, |
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.
Please add necessary comments
@@ -997,17 +992,25 @@ impl TryFrom<types::RefundsResponseRouterData<api::Execute, PaymeRefundResponse> | |||
let refund_status = enums::RefundStatus::try_from(item.response.sale_status.clone())?; | |||
let response = if is_refund_failure(refund_status) { | |||
let payme_response = &item.response; | |||
let status_error_code = payme_response.status_error_code.ok_or( |
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.
Don't validate the field and throw error. Instead if the value is None default error_code, error_message to NO_ERROR_CODE & NO_ERROR_MESSAGE
let status = enums::AttemptStatus::from(item.response.sale_status.clone()); | ||
let response = if is_payment_failure(status) { | ||
let payme_response = &item.response; | ||
let status_error_code = payme_response.status_error_code.ok_or( |
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.
Don't validate the field and throw error. Instead if the value is None default error_code, error_message to NO_ERROR_CODE & NO_ERROR_MESSAGE
…stman-runner * 'main' of github.com:juspay/hyperswitch: (22 commits) chore(version): 2024.02.28.0 chore(postman): update Postman collection files fix(connector): [AUTHORIZEDOTNET] Fix status mapping (#3845) refactor(router): added logs health and deep health (#3780) feat(roles): Change list roles, get role and authorization info api to respond with groups (#3837) fix(core): validate amount_to_capture in payment update (#3830) refactor(connector): [Square] change error message from NotSupported to NotImplemented (#2875) feat(router): add connector mit related columns to the payment methods table (#3764) ci(postman): refactor NMI postman collection (#3805) refactor(connector): [Braintree] Mask PII data (#3759) refactor(connector): [Forte] Mask PII data (#3824) refactor(compatibility): added compatibility layer request logs (#3774) refactor(payment_methods): introduce `locker_id` column in `payment_methods` table (#3760) feat(connector): mask pii information in connector request and response for stripe, aci, adyen, airwallex and authorizedotnet (#3678) chore(version): 2024.02.27.0 fix(core): do not construct request if it is already available (#3826) refactor: incorporate `hyperswitch_interface` into router (#3669) feat: add unique constraint restriction for KV (#3723) feat(connector): [Payme] Add Void flow to Payme (#3817) refactor(connector): [Cybersource] Mask PII data (#3786) ...
Type of Change
Description
Add void to payme for cancelling transaction
Additional Changes
Motivation and Context
How did you test it?
Cannot be tested since cards are failing for Payme
Checklist
cargo +nightly fmt --all
cargo clippy