[MGNLUI-1911] Move back applauncher configuration to ui-admincentral Created: 13/Aug/13 Updated: 03/Oct/13 Resolved: 10/Sep/13 |
|
| Status: | Closed |
| Project: | Magnolia UI |
| Component/s: | applauncher |
| Affects Version/s: | 5.0.1 |
| Fix Version/s: | 5.1 |
| Type: | Task | Priority: | Major |
| Reporter: | Mikaël Geljić | Assignee: | Christoph Meier |
| Resolution: | Fixed | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | 0.5d | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | 0.5d | ||
| Issue Links: |
|
||||||||||||||||||||||||||||||||||||||||||||
| Template: |
|
||||||||||||||||||||||||||||||||||||||||||||
| Acceptance criteria: |
Empty
|
||||||||||||||||||||||||||||||||||||||||||||
| Task DoR: |
Empty
|
||||||||||||||||||||||||||||||||||||||||||||
| Release notes required: |
Yes
|
||||||||||||||||||||||||||||||||||||||||||||
| Date of First Response: | |||||||||||||||||||||||||||||||||||||||||||||
| Sprint: | 5.1 Beta1- Frontend | ||||||||||||||||||||||||||||||||||||||||||||
| Description |
|
appLauncherLayout contentnode will now be in ui-admincentral instead of in ui-framework. As agreed during widgetset concept review
|
| Comments |
| Comment by Mikaël Geljić [ 30/Aug/13 ] |
|
Realized appLauncherLayout config for configuration and security apps is currently bootstrapped in config.modules.ui-framework.config.appLauncherLayout.xml, which should just be the skeleton for the appLauncherLayout, without any app. Please extract these two to their respective modules:
Ensure the ordering is still correct upon fresh install |
| Comment by Christoph Meier [ 30/Aug/13 ] |
|
ok. i did |
| Comment by Christoph Meier [ 03/Sep/13 ] |
|
there are branches on the remote-repo which contain all the changes. Note: at least standard-templating-kit depends on magnolia 5.0.x, yet, but this ticket should be done for 5.1 the google-doc https://docs.google.com/document/d/1Pz5tWdoJD6LkwLdac2fueLoyTH7yXOnsQSTLkEv-rtI/edit gives an overview of the changes which have been made.
although it has not been changed, STKModuleVersionHandlerTest now fails. Might have to do something with the versions ... i'm not quite sure. to review this ticket, use the fix_MGNLUI-1972-branch(es) |
| Comment by Christoph Meier [ 03/Sep/13 ] |
|
since Christopher Z. will review this ticket; i set it to resolved now |
| Comment by Christopher Zimmermann [ 04/Sep/13 ] |
|
Review:
Create tickets in all affected projects and link them to this ticket. |
| Comment by Christoph Meier [ 09/Sep/13 ] |
|
reopened to fix the minor issues coming from review |
| Comment by Christoph Meier [ 09/Sep/13 ] |
|
fixed the minor changes according to the review |
| Comment by Eric Hechinger [ 10/Sep/13 ] |
|
Could you please check your test. Work fin locally but fail on Hudson. In addition could you please simplify this test. |
| Comment by Christoph Meier [ 10/Sep/13 ] |
|
simplified SecurityModuleVersionHandlerTest#testTo51EnsureSecurityAppLauncherConfigOrderAfterExtraInstallTask |
| Comment by Antti Hietala [ 27/Sep/13 ] |
|
Documented in http://documentation.magnolia-cms.com/display/DOCS/App+launcher+layout |