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

VIDEO-8374 - Support for WKWebView/SFSafariViewController #149

Merged
merged 3 commits into from
Feb 11, 2022

Conversation

PikaJoyce
Copy link
Contributor

JIRA: VIDEO-8374

WKWebView Testing Notes

This small PR is for adding support for WKWebView to the SDK.

Contributing to Twilio

All third-party contributors acknowledge that any contributions they provide will be made under the same open-source license that the open-source project is provided under.

  • I acknowledge that all my contributions will be made under the project's license.

'Safari iOS - Slack ',
'Mozilla/5.0 (iPhone; CPU iPhone OS 15_3 like Mac OS X) AppleWebKit/605.1.15 (KHTML, like Gecko) Version/15.3 Mobile/15E148 Safari/604.1',
'safari'
],
[
Copy link
Contributor

Choose a reason for hiding this comment

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

Is one of these user agents from Tim's app?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yes, the Safari-WKWebView - iPhone is from Tim's app.

and Safari iOS - Slack is the same as SFSafariViewController UA from Tim's app so it seems like most of the applications we tested on is using SFSafariViewController.

CHANGELOG.md Outdated
New Features
------------

- Enabled support for WKWebViews.
Copy link
Contributor

Choose a reason for hiding this comment

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

  • Enabled support for WKWebViews. Thank you @codeborne for your contribution.

@PikaJoyce PikaJoyce merged commit 9ac1aac into master Feb 11, 2022
@PikaJoyce PikaJoyce deleted the VIDEO-8374-WKWebView-Support branch February 11, 2022 21:32
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.

2 participants