HomePhabricator
Diffusion Comm cc689095047a

[native] Default to 'natDevHostname' from expo config

Description

[native] Default to 'natDevHostname' from expo config

Summary:
Updated the dev-hostname.js to try gathering the natDevHostname from the Expo config first.
If this fails, then it falls back to the existing facts/network.json flow.

If none is present, the warning will be displayed

Depends on D6267

Test Plan:

  1. Deleted facts/network.json
  2. Opened the app on physical device
  3. No related warning is displayed
  4. Redux devtools are working

Reviewers: ashoat, atul

Reviewed By: ashoat

Subscribers: tomek

Differential Revision: https://phab.comm.dev/D6268

Details

Provenance
bartekAuthored on Jan 14 2023, 8:19 AM
Reviewer
ashoat
Differential Revision
D6268: [native] Default to 'natDevHostname' from expo config
Parents
rCOMM966ee19f20ec: [native] Add 'natDevHostname' field to Expo config
Branches
Unknown
Tags
Unknown