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.

Sign up for GitHub

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

Jump to bottom

Migrate from JavaScript --> TypeScript #6533

Open

spwoodcock opened this issue Aug 20, 2024 · 0 comments · May be fixed by #6534

Assignees

Labels

dev-setup 🛠️ priority: low scope: frontend status: in progress tech-debt 💣

Member

spwoodcock commented Aug 20, 2024 •

edited

Loading

Is your feature request related to a problem? Please describe.

Currently the codebase is JavaScript.
It's a bit messy as we do have some TypeScript declarations inside JS files...
We probably need a big refactor to improve the maintainability of the codebase.
Other issues such as Replace craco / webpack frontend module bundler with vite #6531 are partly blocked by this.

Describe the solution you'd like

Rename .js files to .jsx.
Upgrade all code JavaScript --> TypeScript, with js and jsx to ts and tsx.

Describe alternatives you've considered

JSDoc, but IMO its better for installable packages, not for software tools, where typescript shines.

Additional context

This is low priority, but will be helpful into the future.
@JoltCode has kindly volunteered to look into this.
It's a minor blocker for some other tasks, as it makes sense to do this first.

The text was updated successfully, but these errors were encountered:

spwoodcock added scope: frontend priority: low dev-setup 🛠️ tech-debt 💣 labels

JoltCode linked a pull request

that will close this issue

chore(frontend): migrate to TypeScript and Vite #6534

Draft

3 tasks

spwoodcock assigned JoltCode

JoltCode added the status: in progress label

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment