-
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): [MIFINITY] Implement payment flows and Mifinity payment method #4592
Conversation
…into mifinity-impl
|
||
#[derive(Debug, Default, Serialize, Deserialize)] | ||
pub struct MifinityConnectorMetadataObject { | ||
pub brand_id: String, |
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.
pub brand_id: String, | |
pub brand_id: Secret<String>, |
pub struct MifinityAddress { | ||
address_line1: Secret<String>, | ||
country_code: api_models::enums::CountryAlpha2, | ||
city: String, |
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.
@SamraatBansal Should we consider making this a Secret<String>
?
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.
City and Country are not sensitive info according to pii guidelines.
Core also does not consider it as secret so same is translated here
277f481
0fe7493
…out-fix * 'main' of github.com:juspay/hyperswitch: refactor(connector): convert init payment flow to preprocessing flow for shift4 (#4884) ci(cypress): Add billing address for bank redirects (#4903) refactor(openapi): move openapi to a separate folder (#4859) chore(version): 2024.06.07.0 refactor(outgoing_webhooks): raise errors in the analytics pipeline in case of API client errors or non-2xx responses (#4894) chore(config): [MIFINITY] add configs for Mifinity in WASM (#4895) feat(router): add `acquirer_country_code` in acquirer_details and send it in netcetera authentication flow (#4835) refactor(connector): convert init payment flow to preprocessing flow for nuvei (#4878) feat(connector): [MIFINITY] Implement payment flows and Mifinity payment method (#4592) fix(connectors): [BOA/CYBS] make avs code optional (#4898) feat(events): add metadata info to events (#4875)
Type of Change
Description
Additional Changes
Motivation and Context
How did you test it?
Response
redirection_url
in response complete the paymentpayment_id
Create a payment with "capture_method": "manual",
Response
Checklist
cargo +nightly fmt --all
cargo clippy