-
Notifications
You must be signed in to change notification settings - Fork 0
chore: configure vite to simplify the development #27
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
base: master
Are you sure you want to change the base?
chore: configure vite to simplify the development #27
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
This PR configures Vite for a simplified development workflow by adding environment-based configuration, HTTPS support via mkcert, and proxy settings for local development against an Infinity backend.
Key Changes:
- Vite configuration now validates required environment variables (port and Infinity URL) in development mode
- Added HTTPS support using vite-plugin-mkcert for secure local development
- Configured proxy rules to route webapp3 and API requests to the Infinity backend
Reviewed changes
Copilot reviewed 6 out of 8 changed files in this pull request and generated 4 comments.
Show a summary per file
| File | Description |
|---|---|
| vite.config.ts | Enhanced configuration with environment variable loading, development mode validation, server settings including proxy rules for webapp3 and API endpoints |
| utils/remove-manifest.js | New utility script to delete the built manifest.json file from the dist directory |
| public/manifest.json | Added default manifest configuration with plugin entry pointing to localhost:5173 |
| package.json | Added dependencies for Node.js types and vite-plugin-mkcert |
| README.md | Added documentation for creating and configuring the .env file |
| .env.example | Provided example environment variable configuration |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
d607529 to
c2aa9b8
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
Copilot reviewed 6 out of 8 changed files in this pull request and generated 3 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
80ed5ee to
e9ccec9
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
Copilot reviewed 6 out of 8 changed files in this pull request and generated 3 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
e9ccec9 to
9b95b42
Compare
Closes #21