-
Notifications
You must be signed in to change notification settings - Fork 369
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(payment): PI-3048 [Checkout.com] Remove issuers dropdown and sending BIC #2141
base: master
Are you sure you want to change the base?
Conversation
…ding BIC to backend
@@ -37,7 +41,7 @@ const CheckoutcomCustomPaymentMethod: FunctionComponent<PaymentMethodProps> = ({ | |||
|
|||
const billingAddress = checkoutState.data.getBillingAddress(); | |||
|
|||
if (!isCheckoutcomPaymentMethod(checkoutCustomMethod)) { | |||
if (!isCheckoutcomPaymentMethod(checkoutCustomMethod) || isIdealHostedPageExperimentOn) { |
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 changes to the spec.
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.
Do CheckoutCom has other APMs? If yes, should fields be rendered for them when IdealExperiment is enabled?
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.
Change looks good. LGTM 👍 Can you please update the codeowners file as this shouldn't require checkout review.
What?
iDEAL have updated their central technical infrastructure.
It requires from us to stop calling GET /issuers endpoint on checkout.
The change is under the experiment because according to this comment
we're waiting for official information from Checkoutcom team.
Why?
According to the CheckoutCOM team, the new changes will take effect on April 1, 2025.
We should be informed and then projected rollout of an experiment is 100%.
revert,
disable experiment PI-2979.checkoutcom_enable_ideal_hosted_page
Testing / Proof
Before:
After:
https://github.com/user-attachments/assets/3766b8d8-dc1e-4f72-aa77-e6b8f7ea1caa
@bigcommerce/team-checkout