-
Notifications
You must be signed in to change notification settings - Fork 88
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
Expose web view activation/deactivation in Visitable #135
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.
Nice, thank you!
…ation in the protocol extension.
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.
@svara once the tests pass, you can merge this over and tag a new release.
You may have to bump this timeout for the failing tests when turbo doesn't exist on a page: https://github.com/hotwired/turbo-ios/blob/main/Tests/SessionSpec.swift#L8 (or just use a separate timeout constant for the failing test from the changes here: #133)
…ation also in `VisitableViewController`.
This PR exposes web view activation/deactivation in
Visitable
.The main reason we are exposing them is because we need to delegate the web view activation/deactivation to
Strada
.