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

[POC] Implement SFSafariViewController as fallback when checkoutDidFail #146

Closed
wants to merge 1 commit into from

Conversation

markmur
Copy link
Contributor

@markmur markmur commented Mar 5, 2024

What changes are you making?

POC-SFSafariViewController.1.mp4

Before you merge

Important


Checklist for releasing a new version
  • I have bumped the version number in the podspec file.
  • I have bumped the version number in the README.
  • I have added a Changelog entry.

Tip

See the Contributing documentation for instructions on how to publish a new version of the library.


throw NSError(domain: "com.shopify", code: 1, userInfo: [NSLocalizedDescriptionKey: "Fake Error"])
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Intentionally throwing an error here anytime we receive a bridge event

@markmur markmur force-pushed the markmur/graceful-fallback-poc branch from 502de3c to e51b56f Compare March 5, 2024 14:35
@markmur markmur changed the title Implement SFSafariViewController as fallback when checkoutDidFail [POC] Implement SFSafariViewController as fallback when checkoutDidFail Mar 7, 2024
@markmur markmur self-assigned this Mar 25, 2024
@markmur markmur force-pushed the markmur/graceful-fallback-poc branch 4 times, most recently from 201a903 to 092cb3e Compare March 25, 2024 15:30
@markmur markmur force-pushed the markmur/graceful-fallback-poc branch from 092cb3e to 2e7d1ea Compare March 25, 2024 15:31
@markmur markmur closed this Apr 25, 2024
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.

1 participant