Skip to content
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): PAYPAL-4068 added BT Fastlane address shipping selector #2493

Merged

Conversation

andriiVitvitskyi1990
Copy link
Contributor

@andriiVitvitskyi1990 andriiVitvitskyi1990 commented Apr 30, 2024

What?

Added BT Fastlane address shipping selector

Why?

To implement paypal fastlane shipping address selector

Testing / Proof

Unit tests

Fastlane User

FastLane.User.mov

Non Fastlane User

Screen.Recording.2024-05-17.at.09.13.56.mov

Fastlane Refreshed Page

FastLane.Refreshed.Page.mov

Fastlane Changed To Non Fastlane

FastLane.User.Changed.to.Non.FastLane.mov

Canceled OTP

Canceled.OTP.mov

@bigcommerce/team-checkout @bigcommerce/team-payments

@andriiVitvitskyi1990 andriiVitvitskyi1990 force-pushed the PAYPAL-4068 branch 4 times, most recently from 14731b8 to c412eb3 Compare May 20, 2024 09:09
import { PaymentProviderCustomerActionCreator } from '../../../payment-provider-customer';
import { CardInstrument } from '../../../payment/instrument';
import { UntrustedShippingCardVerificationType } from '../../../payment/instrument/instrument';
import ConsignmentActionCreator from '../../consignment-action-creator';
import { ShippingInitializeOptions, ShippingRequestOptions } from '../../shipping-request-options';
import ShippingStrategy from '../shipping-strategy';
import { BraintreeInitializationData } from '../../../payment/strategies/braintree';
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please check and fix eslint warning

expect(braintreeIntegrationServiceMock.getBraintreeFastlane).toHaveBeenCalled();
});

it('updates customer', async () => {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
it('updates customer', async () => {
it('updates provider customer data', async () => {

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants