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

Update remaining AMD files to ES6 classes. #6975

Closed
shefalijoshi opened this issue Aug 21, 2023 · 1 comment · Fixed by #7029
Closed

Update remaining AMD files to ES6 classes. #6975

shefalijoshi opened this issue Aug 21, 2023 · 1 comment · Fixed by #7029
Labels
help_wanted Help the Open MCT project! type:maintenance tests, chores, or project maintenance verified Tested or intentionally closed
Milestone

Comments

@shefalijoshi
Copy link
Contributor

shefalijoshi commented Aug 21, 2023

Summary

Throughout the codebase, we have a number of files while are still using the AMD pattern. These should be updated to be ES6 classes instead.

@shefalijoshi shefalijoshi added the type:maintenance tests, chores, or project maintenance label Aug 21, 2023
@shefalijoshi shefalijoshi added this to the Target:3.1.0 milestone Aug 21, 2023
@ozyx ozyx added the help_wanted Help the Open MCT project! label Aug 21, 2023
@ozyx ozyx changed the title Find and replace files to be ES6 classes. Update remaining AMD files to ES6 classes. Aug 21, 2023
@depperm depperm mentioned this issue Aug 25, 2023
15 tasks
@akhenry akhenry removed this from the Target:3.1.0 milestone Oct 2, 2023
@unlikelyzero unlikelyzero added this to the Target:3.3.0 milestone Dec 28, 2023
@unlikelyzero unlikelyzero added the verified Tested or intentionally closed label Jan 17, 2024
@unlikelyzero unlikelyzero modified the milestones: Target:3.3.0, Target:4.0.0 Feb 1, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
help_wanted Help the Open MCT project! type:maintenance tests, chores, or project maintenance verified Tested or intentionally closed
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants