Add manual testing instructions #179
Conversation
|
The localtunnel instructions worked this morning from me to set up, run, and use for a PR review. 👍 |
guides/browser-support.md
Outdated
| @@ -114,6 +114,33 @@ browsers: | |||
| - iOS Safari | |||
| - Chrome for Android | |||
|
|
|||
| #### Manual testing | |||
|
|
|||
| We use a mix of Saucelabs and [localtunnel](https://localtunnel.github.io/www/) | |||
guides/browser-support.md
Outdated
|
|
||
| We use a mix of Saucelabs and [localtunnel](https://localtunnel.github.io/www/) | ||
| to manually test our site. Be sure to review both the testing and accessibility | ||
| guides when reviewing sites locally. |
There was a problem hiding this comment.
Should there be links to the guides here or a blurb on where to find them? I feel like someone joining the project could read this and be like "where do I find those?"
guides/browser-support.md
Outdated
|
|
||
| ##### Saucelabs | ||
|
|
||
| _Need Cat's demo instructions._ |
There was a problem hiding this comment.
Maybe don't include this section till Cat's demo instructions are in.
guides/browser-support.md
Outdated
| Start your project's localhost and in a separate tab run `lt --port 8000` | ||
| editing the port value to match your localhost setup. This will create a | ||
| temporary public URL that can be accessed from any device on any connection. | ||
| Navigate to the URL from your device and test to your hearts content, the URL |
guides/browser-support.md
Outdated
| temporary public URL that can be accessed from any device on any connection. | ||
| Navigate to the URL from your device and test to your hearts content, the URL | ||
| will remain active as long as the instance is active (keep this in mind, don't | ||
| leave it running forever). |
There was a problem hiding this comment.
Maybe add to close it with ctrl + c.
jimmynotjim
force-pushed
the
localtunnel-instructions
branch
from
0fd6a56 to
ea08883
Compare
|
|
||
| ##### Sauce Labs | ||
|
|
||
| See [#171](https://github.com/cfpb/development/pull/171) for details on testing |
There was a problem hiding this comment.
This should link to the Pull Request and not the document that 171 added?
There was a problem hiding this comment.
The PR isn't merged yet :( When it is, I'll update to point to that doc
|
It's been more than a week, merging as is and we can update if/when #171 is merged. |
Add manual testing instructions to the browser support guide to help ensure developers are adhering to our list of supported browsers.
Additions
Testing
Todos
Checklist