-
Notifications
You must be signed in to change notification settings - Fork 156
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
Add FluentTester readme for ios #464
Conversation
Adding instructions for how to launch the FluentTester on iOS, following Saad's instructions after recent updates. Will update the main Fluent Tester readme to point to this page.
macOS has essentially the same instructions, so it would be nice to also include that markdown with this one |
Addressed Chris and Saad's feedback
apps/README.md
Outdated
`FluentUI Tester` is the test app that we use to test our FluentUI components during development. It uses the [react-native-test app](https://github.com/microsoft/react-native-test-app) under the covers, and loads the FluentUI bundle. | ||
|
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.
nit: loads the fluent-tester bundle, not FluentUI bundle
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.
Fixed in recent commit.
So is the intention now to keep the documentation for all the apps at apps/README.md instead of having platform specific ones? Or is that just for now while we are only adding iOS? |
I'm doing separate PRs for each platform. For example, here's the macOS one: #465 |
Platforms Impacted
Description of changes
Adding instructions for how to launch the FluentTester on iOS, following Saad's instructions after recent updates. Will update the main Fluent Tester readme to point to this page.
Verification
Ran through the documentation as I was writing it.
Pull request checklist
This PR has considered (when applicable):