-
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
Do not install Bar Chart plugin by default #5088
Comments
@akhenry will this need to be installed in viper-openmct and other downstream projects? |
TestingIs not installed by default in production mode
Is installed by default in development mode
|
@ozyx if someone had an object created with this before the plugin was removed, how would open MCT behave on 'upgrade'? |
Good question. It looks like the current behavior is that any domain objects with missing or unregistered types are still displayed in the object tree, but as objects of Clicking on them routes properly but does not display anything other than the object name in the main view pane. There doesn't seem to be any notification raised that MCT has encountered objects with unregistered types. |
Users will just have to install the plugin to restore the previous behavior, I think it's ok. |
* Do not install Chart plugin by default (#5088) * Install Chart plugin in development mode (#5088) Co-authored-by: Andrew Henry <[email protected]>
I'm not seeing the bar chart under the create menu on testathon. Verified fixed - testathon 06/09/22 |
Summary
The Bar Chart plugin is being installed by default right now. This plugin is still somewhat experimental and in a state of flux, so it should be opt-in for now.
The text was updated successfully, but these errors were encountered: