# Why
Fixes that QR code on web UI pointed to the `Expo Go` directly instead of to the interstitial page.
Fallback to the SDK version if the runtime version wasn't defined.
# How
- Extended the dev graphQL server to serve Interstitial page URL if available.
# Test Plan
- runs the local version of the CLI and confirms that the web QR code points to the correct place.
- checks if the SDK version was detected when the runtime version isn't available.
* [dev-tools] Initialize new Expo app
* [dev-tools] remove decorators
* [dev-tools] get UI rendering via Expo Web instead of NextJS
* [dev-tools] use path.resolve to remove leading slash
* [dev-tools] get Expo Web dev-tools running on `expod start`
* [dev-tools] get local dev server working with `expo start`-ed UI
* [dev-tools] add nodemon watcher for developing DevToolsServer and create script for developing DevTools UI.
This also removes the `yarn start` script, as you should run `yarn dev-server <path>` and `yarn dev-ui` in separate terminals.
* [dev-tools] update README.md
* [dev-tools] build/develop DevTools client using xdl directly
* [dev-tools] add emotion babel plugin dep
* [website] remember to install client dependencies
* [dev-tools] check before cleaning dist path
* [dev-tools] remove nextjs config files
* [dev-tools] test if adding next back to the devtools package unbreaks CI
* Revert "[dev-tools] test if adding next back to the devtools package unbreaks CI"
This reverts commit 3647e51eff44eaddb65c322a66c9728e97b5a0b6.
* [dev-tools] try to add *correct* NextJS back to fix CI
* [dev-tools] use correct start script to not start metro
* [dev-tools] remove redundant main field
we should stop generating this in the template if it's default now
* [dev-tools] remove redundant platforms field
* [dev-tools] moved indexpage to appjs
* Update packages/dev-tools/client/.gitignore
Co-Authored-By: Evan Bacon <[email protected]>
* [dev-tools] be more specific about what version of dev-tools you should be testing
* [dev-toold] reduce size from 4.3mb to 1.2mb (on-par with nextjs predecessor)
* [dev-tools] import name properly
* [dev-tools] tell developers to not run `expo start` when trying to `yarn dev`
* [dev-tools] retake: remove manifest, run on port 19002, and support relative paths
Co-authored-by: Evan Bacon <[email protected]>
This removes SMS from the UI for sending project development links. The server endpoint still behaves the same, but the UI now promotes email over SMS. Email is more reliable, much cheaper, and more customizable. We'll make a similar change on web as well.
Test plan: Ran the CLI on a test project and verified that both the CLI and web UI showed just "email" without "SMS" for sending links.
Łukasz Kosmaty