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

Fix small error in iOS documentation #59

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

chrisortman
Copy link

This fixes a small error in the iOS documentation that I ran into. The old documentation stated that the bridge components must be registered before routing your first URL. However, because the userAgent is checked to determine if we are in a Hotwire app and therefore if the bridge components should be activated and because this userAgent is established when the root view controller is created you need to register your bridge components first.

This fixes a small error in the iOS documentation that I ran into. The old documentation stated that the bridge components must be registered before routing your first URL. However, because the userAgent is checked to determine if we are in a Hotwire app and therefore if the bridge components should be activated **and** because this userAgent is established when the root view controller is created you need to register your bridge components first.
@ibrahima
Copy link

FWIW there is a PR that might fix this issue, but it would be really helpful to update the docs in the meantime so that people don't run into it.

@chrisortman
Copy link
Author

Thanks for pointing that PR out. It looks like my description is not fully correct. I assumed that it was the rootviewcontroller creation but it is really the Navigator. Looks like the fix will address the root issue

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants