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

Migrate c-zoom.vue to TypeScript #1963

Closed
Tracked by #1936
ckcherry23 opened this issue Mar 26, 2023 · 0 comments · Fixed by #1965
Closed
Tracked by #1936

Migrate c-zoom.vue to TypeScript #1963

ckcherry23 opened this issue Mar 26, 2023 · 0 comments · Fixed by #1965
Assignees

Comments

@ckcherry23
Copy link
Member

ckcherry23 commented Mar 26, 2023

Part of #1936

Migrate c-zoom.vue to TypeScript

@ckcherry23 ckcherry23 self-assigned this Mar 26, 2023
@ckcherry23 ckcherry23 changed the title c-zoom.vue Migrate c-zoom.vue to TypeScript Mar 26, 2023
@sikai00 sikai00 moved this to For contributors in RepoSense Roadmap Mar 28, 2023
dcshzj pushed a commit that referenced this issue Apr 10, 2023
Currently, despite the addition of TypeScript support, the frontend is
still largely written in JavaScript. This results in a lack of type
safety and many complex objects being passed around as unknown types,
which may in turn lead to errors.

Let's migrate one of the main components, c-zoom.vue, to TypeScript.
In addition to providing greater type safety, the migration of this
component will also allow the migration of its child components, such
as c-ramp.vue.
@github-project-automation github-project-automation bot moved this from For contributors to Closed/Completed in RepoSense Roadmap Apr 10, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Closed/Completed
Development

Successfully merging a pull request may close this issue.

1 participant