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

This fixes onError not propagated to grandparent of the window #349

Merged
merged 2 commits into from
Oct 10, 2024

Conversation

ayinloya
Copy link
Collaborator

@ayinloya ayinloya commented Oct 10, 2024

In smile links, the onError is not called due to the grandparent of the window not receiving the consent denied message.

This also adds optional chaining to the legacy product constraint check.

Testing

run the example up
Select enhanced or basic kyc
Select NG then bvn
Cancel the consent
The screen should close

@ayinloya ayinloya requested a review from solnsubuga October 10, 2024 12:26
@ayinloya ayinloya self-assigned this Oct 10, 2024
Copy link

This branch has been deployed to s3 / cloudfront.

✅ Preview URL for Smart Camera Web:
https://cdn.smileidentity.com/js/preview-fix-error-not-propagated/smart-camera-web.js
✅ Preview URL for Embed:
https://cdn.smileidentity.com/inline/preview-fix-error-not-propagated/js/script.min.js

Copy link
Contributor

@beastawakens beastawakens left a comment

Choose a reason for hiding this comment

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

LGTM

@ayinloya ayinloya merged commit 39ae5c9 into main Oct 10, 2024
10 checks passed
@ayinloya ayinloya deleted the fix-error-not-propagated branch October 10, 2024 21:15
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