-
-
Notifications
You must be signed in to change notification settings - Fork 29
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
Instructions for mobile device testing on a branch #1479
Conversation
fe729de
to
265a753
Compare
I would like to get @longrunningprocess review before this is merged. |
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.
I'm not quite on board with this yet, I need to better understand. Maybe we can get together and discuss "in-person" first.
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.
I really like those updates to the docs, thanks @megahirt , just one final question about the necessity of that platform
line
We can use ngrok to create a URL that accesses what's in localhost from another device.
This allows us to access the unencrypted HTTP localhost if we want to, which is useful when used with ngrok
cc01515
to
d92686e
Compare
also add a TOC heading for mobile device testing
f3a32d2
to
6203f98
Compare
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.
This is ready to merge
After cleaning up this PR, this is good to go. Just documentation change with the approved disable_https_redirect
Description
We can use ngrok to create a URL that accesses what's in localhost from another device. This allows us to test local branches' functionality on mobile devices. This PR adds instructions about how to do this to Developer.md.
Also, we disable HTTPS redirects in Caddy to facilitate easy ngrok usage
Fixes #1478
Type of Change
-Expanding developer instructions
Testing on your branch
N/A
Checklist
qa.languageforge.org testing
Reviewers: add/replace your name below and check the box to sign-off/attest the feature works as expected on qa.languageforge.org