-
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): [Iatapay] add upi qr support #4728
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.
LGTM!
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.
Routing specific changes LGTM
crates/openapi/src/openapi.rs
Outdated
@@ -308,6 +310,7 @@ Never share your secret api keys. Keep them guarded and secure. | |||
api_models::payments::ApplepayConnectorMetadataRequest, | |||
api_models::payments::SessionTokenInfo, | |||
api_models::payments::SwishQrData, | |||
api_models::payments::UpiData, |
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.
UpiData is already there
a81a6a3
1df258f
pub enum UpiData { | ||
UpiCollect(UpiCollectData), | ||
UpiIntent(UpiIntentData), | ||
} | ||
|
||
#[derive(Debug, Clone, Eq, PartialEq, serde::Deserialize, serde::Serialize)] | ||
#[serde(rename_all = "snake_case")] | ||
pub struct UpiCollectData { |
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.
we have changed a struct to enum, is this change backwards compatible? If this breaks existing api contract then this change cannot be made.
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.
Through stripe compatibility layer, backwards compatibility is handled.
ref - https://github.com/juspay/hyperswitch/pull/4728/files#diff-b3a18db4870f5c320bcaeedfbdaeec5c2270e75850d20f47f4ef2adaeffeafe9
…actor * 'main' of github.com:juspay/hyperswitch: (39 commits) Fix(Cypress): Fixing 3DS payment failure in headless mode (#4807) feat(users): Add redis in Begin and Verify TOTP and create a new API that updates TOTP (#4765) refactor(connector): [Klarna] Refactor Authorize call and configs for prod (#4750) fix: implement StrongEq for Vec<u8> (#4795) refactor(core): move router data response and request models to hyperswitch domain models crate (#4789) chore(version): 2024.05.29.0 refactor: retrieve extended card info config during business profile get call (#4784) refactor(payment_methods): add support for passing ttl to locker entries (#4690) feat(connector): [CRYPTOPAY] Pass network details in payment request (#4779) fix: include client columns in payment attempts response struct (#4761) fix(docker-compose): fix docker compose syntax (#4782) feat(connector): [Iatapay] add upi qr support (#4728) docs(analytics): Add documentation for setting up data services and enabling data features in control center (#4741) chore(version): 2024.05.28.0 feat(connector): [AUTHORIZEDOTNET] Implement non-zero mandates (#4758) feat(core): [Paypal] Add session_token flow for Paypal sdk (#4697) chore(version): 2024.05.27.0 chore: add missing migrations for recently added currencies (#4760) Refactor(core): Inclusion of constraint graph for merchant Payment Method list (#4626) chore(version): 2024.05.24.1 ...
Type of Change
Description
This PR introduces a new payment_method_type:
upi_intent
and a new payment_method_data:upi_qr: {}
along with iatapay qr code implementation.Additional Changes
How did you test it?
Note:Test through stripe compatibility layer
Response
Response
Stimulate Terminal status (for Failed -
FAILED
)Checklist
cargo +nightly fmt --all
cargo clippy