Skip to content
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

Reapply "Switch from create-react-app to vite (#7808)" with fixes #7828

Merged
merged 1 commit into from
Sep 23, 2024

Conversation

joechung-msft
Copy link
Contributor

@joechung-msft joechung-msft commented Sep 19, 2024

This reverts commit a1dfb00.

Changes:

  • Remove basePlugin and customize base configuration:

    • Set base to / instead of ./
  • Remove buildPathPlugin and customize build configuration:

    • Empty build folder before each build to ensure clean builds
    • Rename assets to static to align with what AFD expects
    • Rename dist to build

@joechung-msft joechung-msft changed the title Reapply "Switch from create-react-app to vite (#7808)" (#7824) Reapply "Switch from create-react-app to vite (#7808)" with fixes Sep 19, 2024
This reverts commit a1dfb00.

Additional changes:

- Remove `basePlugin` and customize base configuration:
  - Set base to `/` instead of `./`.

- Remove `buildPathPlugin` and customize build configuration:
  - Empty `build` folder before each build to ensure clean builds.
  - Rename `assets` to `static` to align with what AFD expects.
  - Rename `dist` to `build`.
@joechung-msft joechung-msft merged commit 0d3dab1 into Azure:dev Sep 23, 2024
7 checks passed
@joechung-msft joechung-msft deleted the joechung/vite-take-two branch September 23, 2024 17:32
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