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

Feature/vite #975

Open
wants to merge 4 commits into
base: develop
Choose a base branch
from
Open

Feature/vite #975

wants to merge 4 commits into from

Conversation

CourtneyGosselin
Copy link
Member

Feature Vite

Description

This PR migrates the project from react-scripts to Vite, as react-scripts is now deprecated.

Expected Behaviour

The portal should function as before, with improved build performance and faster development reloads.

Associated PRs

Data portal

Signed-off-by: Courtney Gosselin <courtney@gosselin.io>
Signed-off-by: Courtney Gosselin <courtney@gosselin.io>
@OrdiNeu
Copy link
Contributor

OrdiNeu commented Feb 24, 2025

Can you rebase this to develop or merge develop into this branch? Running into "There was a problem proxying the request" and unsure if it's due to some incompatibility between the branches or not.

@OrdiNeu
Copy link
Contributor

OrdiNeu commented Feb 26, 2025

The Playwright tests aren't passing for some reason -- Vite appears to not work with it?

image

image

It gets stuck on a whitescreen

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