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

chore: make UI dev easier #1478

Merged
merged 9 commits into from
Apr 10, 2023
Merged

chore: make UI dev easier #1478

merged 9 commits into from
Apr 10, 2023

Conversation

markphelps
Copy link
Collaborator

@markphelps markphelps commented Apr 10, 2023

This PR changes index.html for the UI to be a template that is rendered differently if we are building for 'release' mode or 'development' mode.

If in development mode, it proxies all UI requests to the npm dev server which allows for hot reloading via vite: https://vitejs.dev/guide/backend-integration.html

Also updates to include better instructions on what to do when running mage or mage dev

CleanShot 2023-04-10 at 09 37 51

@markphelps markphelps requested a review from a team as a code owner April 10, 2023 13:44
@@ -1,13 +1,28 @@
<!DOCTYPE html>
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

this is simply to get unit tests/etc to pass as our build expects this file to be here. it will get overwritten during the mage or mage dev tasks and is added to .gitignore so any changes will not be committed to git

@codecov
Copy link

codecov bot commented Apr 10, 2023

Codecov Report

Merging #1478 (fc81852) into main (be09e4e) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main    #1478   +/-   ##
=======================================
  Coverage   77.77%   77.77%           
=======================================
  Files          44       44           
  Lines        3402     3402           
=======================================
  Hits         2646     2646           
  Misses        603      603           
  Partials      153      153           

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

Copy link
Contributor

@yquansah yquansah left a comment

Choose a reason for hiding this comment

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

LGTM

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