resolve comment: make stripe country req

juspay · Jan 18, 2024 · 6d65743 · 6d65743
1 parent 1717524
commit 6d65743
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 4 deletions.
diff --git a/crates/router/src/connector/aci/transformers.rs b/crates/router/src/connector/aci/transformers.rs
@@ -204,7 +204,7 @@ impl
                     payment_brand: PaymentBrand::Sofortueberweisung,
                     bank_account_country: Some(country.to_owned().ok_or(
                         errors::ConnectorError::MissingRequiredField {
-                            field_name: "country",
+                            field_name: "sofort.country",
                         },
                     )?),
                     bank_account_bank_name: None,

diff --git a/crates/router/src/connector/paypal/transformers.rs b/crates/router/src/connector/paypal/transformers.rs
@@ -367,7 +367,7 @@ fn get_payment_source(
                 })?
                 .get_billing_name()?,
             country_code: country.ok_or(errors::ConnectorError::MissingRequiredField {
-                field_name: "sofort.billing_details.country",
+                field_name: "sofort.country",
             })?,
             experience_context: ContextStruct {
                 return_url: item.request.complete_authorize_url.clone(),

diff --git a/crates/router/src/connector/stripe/transformers.rs b/crates/router/src/connector/stripe/transformers.rs
@@ -292,7 +292,7 @@ pub struct StripeSofort {
     #[serde(rename = "payment_method_options[sofort][preferred_language]")]
     preferred_language: Option<String>,
     #[serde(rename = "payment_method_data[sofort][country]")]
-    country: Option<api_enums::CountryAlpha2>,
+    country: api_enums::CountryAlpha2,
 }
 
 #[derive(Debug, Eq, PartialEq, Serialize)]
@@ -1648,7 +1648,9 @@ impl TryFrom<&payments::BankRedirectData> for StripePaymentMethodData {
             } => Ok(Self::BankRedirect(StripeBankRedirectData::StripeSofort(
                 Box::new(StripeSofort {
                     payment_method_data_type,
-                    country: *country,
+                    country: country.ok_or(errors::ConnectorError::MissingRequiredField {
+                        field_name: "sofort.country",
+                    })?,
                     preferred_language: preferred_language.clone(),
                 }),
             ))),