[MGNLUI-2232] No "import" action available on root node in Pages app Created: 03/Oct/13 Updated: 15/Nov/13 Resolved: 17/Oct/13 |
|
| Status: | Closed |
| Project: | Magnolia UI |
| Component/s: | pages app |
| Affects Version/s: | 5.1 |
| Fix Version/s: | 5.1.1 |
| Type: | Bug | Priority: | Major |
| Reporter: | Andreas Weder | Assignee: | Cheng Hu |
| Resolution: | Fixed | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Environment: |
5.1 RC2 Enterprise Edition (fresh installation) |
||
| Attachments: |
|
||||||||
| Issue Links: |
|
||||||||
| Template: |
|
||||||||
| Acceptance criteria: |
Empty
|
||||||||
| Task DoD: |
[ ]*
Doc/release notes changes? Comment present?
[ ]*
Downstream builds green?
[ ]*
Solution information and context easily available?
[ ]*
Tests
[ ]*
FixVersion filled and not yet released
[ ] 
Architecture Decision Record (ADR)
|
||||||||
| Bug DoR: |
[ ]*
Steps to reproduce, expected, and actual results filled
[ ]*
Affected version filled
|
||||||||
| Date of First Response: | |||||||||
| Sprint: | 5.1.1 | ||||||||
| Description |
|
There's no "import" action in Pages app, if no page and thus implicitly the root node is selected (see screen shot). We should offer "import" here, however, to be able to e.g. import the demo project. |
| Comments |
| Comment by Daniel Lipp [ 17/Oct/13 ] |
|
Just a tiny thing: to be 100% correct we should check for the precise path of the node we want to add the property to - not just for it's parent:
register(DeltaBuilder.update("5.1.1", "")
.addTask(new NodeExistsDelegateTask("Add root availability to import", "Add root availability to import action in Pages app", RepositoryConstants.CONFIG, "/modules/pages/apps/pages/subApps/browser/actions/import/availability",
new NewPropertyTask("Add root availability to import", "Add root availability to import action in Pages app", RepositoryConstants.CONFIG, "/modules/pages/apps/pages/subApps/browser/actions/import/availability", "root", true)))
|