Skip to content

Feat/added ppcp fastlane 3ds #2884

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

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

andriiVitvitskyi1990
Copy link
Contributor

@andriiVitvitskyi1990 andriiVitvitskyi1990 commented May 30, 2025

What?

Added 3ds check for PPCP FL

Why?

To be able to check PPCP FL payment flow with 3DS validation

Testing / Proof

Screen.Recording.2025-06-11.at.15.32.11.mov
Screen.Recording.2025-06-11.at.15.29.29.mov

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

@bcsnyk
Copy link
Contributor

bcsnyk commented May 30, 2025

🎉 Snyk checks have passed. No issues have been found so far.

code/snyk check is complete. No issues have been found. (View Details)

@andriiVitvitskyi1990 andriiVitvitskyi1990 force-pushed the feat/added_ppcp_fastlane_3ds branch from 865e7db to 1f7ad64 Compare May 30, 2025 16:16
@andriiVitvitskyi1990 andriiVitvitskyi1990 force-pushed the feat/added_ppcp_fastlane_3ds branch from 1f7ad64 to 770cc5e Compare May 30, 2025 16:23
@andriiVitvitskyi1990 andriiVitvitskyi1990 force-pushed the feat/added_ppcp_fastlane_3ds branch 4 times, most recently from e105db3 to 2e1e933 Compare June 11, 2025 12:47
@andriiVitvitskyi1990 andriiVitvitskyi1990 marked this pull request as ready for review June 12, 2025 09:50
@andriiVitvitskyi1990 andriiVitvitskyi1990 requested a review from a team as a code owner June 12, 2025 09:50
@andriiVitvitskyi1990 andriiVitvitskyi1990 force-pushed the feat/added_ppcp_fastlane_3ds branch 5 times, most recently from affa546 to 417cc02 Compare June 12, 2025 12:55
}

if (authenticationState === 'canceled') {
console.log('3DS check was canceled');
Copy link
Contributor

Choose a reason for hiding this comment

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

console.error might be better

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Fixed

Fastlane(options?: PayPalFastlaneOptions): Promise<PayPalFastlane>;
}

interface ThreeDomainSecureClientShowResponse {
liabilityShift: string;
authenticationState: string;
Copy link
Contributor

Choose a reason for hiding this comment

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

Would be better to set enum type up here with available states instead of string. The same action for liabilityShift and use these enums where needed

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Fixed

@andriiVitvitskyi1990 andriiVitvitskyi1990 force-pushed the feat/added_ppcp_fastlane_3ds branch 2 times, most recently from 654ebbb to b5a742e Compare June 13, 2025 09:52
@andriiVitvitskyi1990 andriiVitvitskyi1990 force-pushed the feat/added_ppcp_fastlane_3ds branch from b5a742e to 8370b7a Compare June 13, 2025 10:01
@andriiVitvitskyi1990 andriiVitvitskyi1990 force-pushed the feat/added_ppcp_fastlane_3ds branch from 8370b7a to 2d8d886 Compare June 13, 2025 10:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants