-
Notifications
You must be signed in to change notification settings - Fork 33
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
3ds - onClose #103
base: main
Are you sure you want to change the base?
3ds - onClose #103
Conversation
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.
Was this tested to confirm that the onError callback is still executed? I took a look at the ticket and it wasn't clear what the root cause of the issue is.
I think that this conditional will not execute onError
if result.success = false
which was fixed in ticket DTPPCPSDK-2207.
@mchoun I think that |
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.
🎉
I think the best way to test that After looking at this fix we made here, it's actually not the |
Based on my understanding of the slack thread, |
Calling onClose for 3Ds to fix issue for this ticket (LI-63320).