-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
Remove all non-legacy usage of Zepto #4698
Comments
We should also remove exports-loader as a part of this PR |
Testing Notes (taken from #5159 (comment))Plugins affected:
To test, all functionality of the listed plugins should remain the same prior to these changes.
|
Note that we may need to temporarily re-enable the creation of summary widgets to test this. |
We need to enable summary widgets to test this issue. |
Will need to create a local viper-openmct install based on release/2.0.5 with summary widgets enabled |
Verified fixed by importing attached JSON file that includes a summary widget. No need to enable creation of summary widgets |
Is your feature request related to a problem? Please describe.
Zepto is a performant drop-in replacement for JQuery and Angular's $ function. It was used pretty extensively in our legacy codebase, but there is also some usage in our current codebase.
Describe the solution you'd like
All usage of zepto in our current codebase (ie. everything under
/src/
) should be replaced with native JavaScript API calls.The following usage of Zepto in non-deprecated code shoulld be removed:
Additional context
There is also usage of Zepto in our legacy codebase (ie. under "platform"), and in Summary Widgets but this code is deprecated and will be removed under a separate task.
The text was updated successfully, but these errors were encountered: