Need to retain the same file structure so that pluggins continue to work.
Basically we have existing pluggins that use this file structure, we need to honor it.
This is not directly related to what we are talking about, but it does mean that we need to move static files out of
/openstack_dashboard/static/MYDASHBOARD/* and into
Auto-discovery of static resources will need to also honor the pluggin model above, hence the file structure above.
You will still need to manually define the ADD_ANGULAR_MODULES in your enabled file, auto-discovery doesn't know what you want enabled.
Sean's patch is going to do that, but having some issues with SCSS.
hz.dashboard module will be empty because the hz.dashboard.MYDASHBOARD module will live at the app level via
ADD_ANGULAR_MODULES. I would argue that it makes no sense to have an empty module, my preference is to just delete it.
Constants are globally available in the app, something I think actually should be avoided, not encouraged.
Having hz.dashboard.tech-debt and workflow in the enabled file is not correct.
They are core components needed by all dashboards and should be loaded by default, not via the pluggin mechanism.
Lets say I have my own dashboard call MYDASHBOARD, and I decided to disable all other dashboards except mine,
all of a sudden, things will break horribly because tech-debt and workflow are not loaded. I would have to either:
a. load the _10_project enabled file
b. copy/paste over the dependencies from _10_project
Furthermore, if I have hz.dashboard module, where do I load that, in _10_project or _x_MYDASHBOARD?
Same issue when I disable entire dashboards.
Tyr's patch will address this problem by having a core module.
-----Richard Jones <r1chardj0...@gmail.com> wrote: -----
__________________________________________________________________________ OpenStack Development Mailing List (not for usage questions) Unsubscribe: openstack-dev-requ...@lists.openstack.org?subject:unsubscribe http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev