Skip to content

[website] Remove chalet for local dev#682

Merged
byronkarlen merged 1 commit into
mainfrom
byron/_website_remove_chalet_for_local_dev
May 11, 2026
Merged

[website] Remove chalet for local dev#682
byronkarlen merged 1 commit into
mainfrom
byron/_website_remove_chalet_for_local_dev

Conversation

@byronkarlen
Copy link
Copy Markdown
Contributor

@byronkarlen byronkarlen commented May 8, 2026

Why

Related to https://github.com/expo/universe/pull/27322. Chalet is no longer needed.

How

  • Remove chalet from dependencies
  • Update docs
  • Remove website from snack-proxies. Expo developers should set up universe first to configure https://snack.expo.test for local dev

Test Plan

This only affects local dev (which is currently broken). Local dev is fixed with this and the upstack PR.

Copy link
Copy Markdown
Contributor Author

byronkarlen commented May 8, 2026

@byronkarlen byronkarlen changed the base branch from byron/tsconfig_and_eslint_cleanup to graphite-base/682 May 8, 2026 21:28
@byronkarlen byronkarlen force-pushed the byron/_website_remove_chalet_for_local_dev branch from 43ef2c6 to c8d571f Compare May 8, 2026 23:00
@byronkarlen byronkarlen force-pushed the graphite-base/682 branch from e7e34f6 to 92ad220 Compare May 8, 2026 23:00
@byronkarlen byronkarlen changed the base branch from graphite-base/682 to byron/tsconfig_and_eslint_cleanup May 8, 2026 23:00
@byronkarlen byronkarlen marked this pull request as ready for review May 8, 2026 23:51
@byronkarlen byronkarlen requested a review from byCedric as a code owner May 8, 2026 23:51
@byronkarlen byronkarlen requested a review from ide May 8, 2026 23:52
Comment thread website/README.md Outdated
Comment thread website/README.md Outdated

Start the Expo website. `snack-proxies` automatically detects the local server and routes all trafic to localhost:3001 when possible.
When testing authentication, it is important that the chalet `expo.test` domain is used, otherwise authentication credentials cannot be accessed by `snack.expo.test`.
Start the Expo website. Required for testing authenticated flows.
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The disclaimer about the test domain still applies right?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes can update to be clearer.

@byronkarlen byronkarlen force-pushed the byron/_website_remove_chalet_for_local_dev branch from c8d571f to 6de8d23 Compare May 9, 2026 00:26
@byronkarlen byronkarlen changed the base branch from byron/tsconfig_and_eslint_cleanup to graphite-base/682 May 11, 2026 15:39
@byronkarlen byronkarlen force-pushed the byron/_website_remove_chalet_for_local_dev branch from 6de8d23 to 3b09c31 Compare May 11, 2026 15:40
@byronkarlen byronkarlen changed the base branch from graphite-base/682 to main May 11, 2026 15:40
@byronkarlen byronkarlen merged commit 1482c12 into main May 11, 2026
35 of 61 checks passed
@byronkarlen byronkarlen deleted the byron/_website_remove_chalet_for_local_dev branch May 11, 2026 16:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants