-
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): [Bambora APAC] add mandate flow #5376
Conversation
@@ -30,7 +30,7 @@ airwallex.base_url = "https://api-demo.airwallex.com/" | |||
applepay.base_url = "https://apple-pay-gateway.apple.com/" | |||
authorizedotnet.base_url = "https://apitest.authorize.net/xml/v1/request.api" | |||
bambora.base_url = "https://api.na.bambora.com" | |||
bamboraapac.base_url = "https://demo.ippayments.com.au/interface/api/dts.asmx" | |||
bamboraapac.base_url = "https://demo.ippayments.com.au/interface/api" |
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.
Should we do this change for prod baseUrl?
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.
zero auth mandate requires different endpoint both in sandbox and production i.e sipp.asmx thus this change was made
.submit_single_payment_result | ||
.response | ||
.credit_card_token | ||
.ok_or(errors::ConnectorError::MissingConnectorMandateID)?; |
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.
Move this error to recurring mandate flow.
.token | ||
.map(|token| -> types::MandateReference { | ||
types::MandateReference { | ||
connector_mandate_id: Some(token), | ||
payment_method_id: None, | ||
} |
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.
Throw missing mandate id error for failure case
) -> CustomResult<RequestContent, errors::ConnectorError> { | ||
let connector_req = bamboraapac::get_setup_mandate_body(req)?; | ||
|
||
Ok(RequestContent::RawBytes(connector_req)) |
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.
For longer term though, we may want to use a different variant specific for SOAP requests.
* 'main' of github.com:juspay/hyperswitch: refactor(router): remove `connector_account_details` and `connector_webhook_details` in merchant_connector_account list response (#5457) feat: add env variable for enable key manager service (#5442) chore(version): 2024.07.29.0 feat(payments): support sort criteria in payments list (#5389) refactor(connector): add amount conversion framework to placetopay (#4988) feat(connector): [Bambora APAC] add mandate flow (#5376) ci: set code owners for payment methods files (#5453) refactor(opensearch): Add Error Handling for Empty Query and Filters in Request (#5432) chore: address Rust 1.80 clippy lints (#5447) feat(connector): [FISERV] Move connector to hyperswitch_connectors (#5441) ci: add support for just hack_v2 check for PRs (#5426) fix: added created at and modified at keys in PaymentAttemptResponse (#5412) refactor(merchant_account_v2): recreate id for `merchant_account` v2 (#5439) chore(version): 2024.07.26.0 feat(events): forward the tenant configuration as part of the kafka message (#5224) refactor(connector): add amount conversion framework to payone (#4981) refactor(user_roles): make org and merchant id nullable (#5353) fix(euclid): change the address taken in SessionFlowRouting from shipping to billing address (#5435)
Type of Change
Description
CIT (Setup mandate flow and mandate creation with payment) is added for bambora apac. MIT is also implementated
Additional Changes
Motivation and Context
How did you test it?
Tested through Postman:
Checklist
cargo +nightly fmt --all
cargo clippy