Skip to content

Beginning rework eendraadschema to be compliant with Vite framework and ESM modules #9

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

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

verschaevesiebe
Copy link

A pullrequest to transfer simple plain html & javascript to Vite.

This brings a lot of advantages where we can provide a way to seperate concerns and implementations as currently the application has loads of typescript files that all share the same "global variables on a window level".

I've replicated the same behaviour through the use of the window object within ESM module.
I propose to thoroughly test the application out (normally everything works except for the situatieplan as this functionality isn't included, why ? )

Please let me know comments or if there's questions we can always contact eachother.

Siebe Verschaeve added 2 commits April 29, 2025 14:33
- Implemented LocalStorageService class to handle saving, retrieving, adding, updating, and deleting items in localStorage.
- Added methods for managing arrays of objects with error handling for localStorage operations.

chore: initialize TypeScript configuration

- Created tsconfig.json with settings for ES2020, module resolution, and linting options.
- Configured TypeScript to include the src directory for compilation.
…ettings

- Integrated vite-plugin-javascript-obfuscator for code obfuscation during build
- Configured Terser for aggressive minification, including:
  - Mangle top-level variable names
  - Remove console and debugger statements
  - Strip all comments from the output
- Set build options to empty output directory before each build
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.

1 participant