fix(cli): only show web ready
in expo start
when enabled
#26694
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Why
Fixes #26519
With Metro being the default (web) bundler, the "Web is waiting on ..." message incorrectly popped up (before web dependencies are installed, and opt-in through
expo.platforms: [..., 'web']
.When opening the link, without actually enabling web support, users received the Expo Go manifest.
How
DevServerManagerActions
(by checkingthis.options.platforms?.includes('web')
Test Plan
$ bun create expo ./test --template blank@50
$ cd ./test
$ expod start
Should see:
$ bun expo install react-native-web react-dom @expo/metro-runtime
$ expod start
Checklist
npx expo prebuild
& EAS Build (eg: updated a module plugin).