-
Notifications
You must be signed in to change notification settings - Fork 63
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: added support for ClickToPay #846
base: main
Are you sure you want to change the base?
Conversation
// Cookie helpers | ||
@val external document: {..} = "document" | ||
|
||
let setLocalStorage = (~key: string, ~value: 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.
Hey @seekshiva, for ClickToPay should we store the recognitionToken in the cookies or in localStorage? Both scenarios will work in Chrome browser however cause issue in Safari
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.
what works in safari?
fcae637
to
77c3d9b
Compare
77c3d9b
to
f16263a
Compare
f16263a
to
651bc73
Compare
651bc73
to
6643d24
Compare
Type of Change
Description
Added support for ClickToPay - Doc Link
How did you test it?
Tested Basic ClickToPay flows which are supported by Mastercard - Test Case Sheet
Checklist
npm run re:build