You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Is your feature request related to a problem? Please describe.
We need to be using our own API functions when possible. A current example of where this needs to be done, is using PreviewAction.js instead of our ActionsAPI.
Is your feature request related to a problem? Please describe.
We need to be using our own API functions when possible. A current example of where this needs to be done, is using PreviewAction.js instead of our ActionsAPI.
This is a general search for where it's being misused (ignoring it's own plugin)
https://github.com/search?q=repo%3Anasa%2Fopenmct%20import%20PreviewAction&type=code
Describe the solution you'd like
Replace direct usage of the PreviewAction.js class and use the ActionsAPI version.
Describe alternatives you've considered
The alternative is what's currently being done, we need to get away from that.
Additional context
Example:
Not using API
https://github.com/nasa/openmct/blob/master/src/plugins/LADTable/components/LadRow.vue#L64
https://github.com/nasa/openmct/blob/master/src/plugins/LADTable/components/LadRow.vue#L239-L273
Using the API we can remove the import and replace this line: https://github.com/nasa/openmct/blob/master/src/plugins/LADTable/components/LadRow.vue#L239
with
The text was updated successfully, but these errors were encountered: