Skip to content

Migrate to Vite#305

Merged
dsternlicht merged 10 commits intodsternlicht:masterfrom
v1lling:feature/vite-migration
Apr 1, 2026
Merged

Migrate to Vite#305
dsternlicht merged 10 commits intodsternlicht:masterfrom
v1lling:feature/vite-migration

Conversation

@v1lling
Copy link
Copy Markdown
Contributor

@v1lling v1lling commented Sep 24, 2025

Related to Issue #303

The build system was migrated from react-scripts to Vite in order to resolve security vulnerabilities and improve build performance.

  • All npm vulnerabilities were eliminated.
  • Build times and hot reload are significantly faster.
  • Modern, actively maintained tooling.
  • Reduced bundle size.

Breaking changes:
The Config.js format: JavaScript configuration files must now use [...]
window.config = {... ';
instead of
export default {...} }

The location remains the same (/public/config.js).
JSON configuration files (config.json) remain unaffected.
The README has been updated with migration instructions.

@dsternlicht
It works in my environment and my RESTool setup (config.js), but users who have made custom modifications may encounter additional issues. This is a suggestion for the project. Whether to merge this migration or handle it differently is up to you.

@v1lling
Copy link
Copy Markdown
Contributor Author

v1lling commented Dec 18, 2025

Hi @dsternlicht 👋
Quick check-in on this one - anything I can do to help get it merged? If the breaking changes are a concern and you'd rather not include it, totally fine too - just good to know either way.

@v1lling
Copy link
Copy Markdown
Contributor Author

v1lling commented Mar 25, 2026

Hey @dsternlicht 👋 Just wanted to bump this in case it got lost, would love to get some feedback when you have a moment. Happy to make any changes needed to get this merged. Thanks!

@dsternlicht
Copy link
Copy Markdown
Owner

@v1lling can you add a migration MD file for this?

Comment thread README.md Outdated
@dsternlicht dsternlicht merged commit ecb0080 into dsternlicht:master Apr 1, 2026
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