[CONTTAGS-42] Remove dependency to personalization Created: 29/Sep/17 Updated: 20/Oct/17 Resolved: 12/Oct/17 |
|
| Status: | Closed |
| Project: | Content Tags |
| Component/s: | None |
| Affects Version/s: | None |
| Fix Version/s: | 1.0 |
| Type: | Task | Priority: | Neutral |
| Reporter: | Christoph Meier | Assignee: | Roman Kovařík |
| Resolution: | Fixed | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Issue Links: |
|
||||||||||||||||||||
| Template: |
|
||||||||||||||||||||
| Acceptance criteria: |
Empty
|
||||||||||||||||||||
| Task DoR: |
Empty
|
||||||||||||||||||||
| Date of First Response: | |||||||||||||||||||||
| Epic Link: | Tagging | ||||||||||||||||||||
| Sprint: | Kromeriz 116, Kromeriz 117, Kromeriz 118 | ||||||||||||||||||||
| Story Points: | 3 | ||||||||||||||||||||
| Description |
|
As an EE-Pro customer I would like to use content-tags independent from p13n. |
| Comments |
| Comment by Milan Divilek [ 09/Oct/17 ] |
|
Reopen:
<components>
<id>subapp</id>
<component>
<type>info.magnolia.contenttags.shortcut.TagShortcutAwareBrowserPresenter</type>
<implementation>info.magnolia.contenttags.shortcut.TagShortcutAwareBrowserPresenter</implementation>
</component>
<component>
<type>info.magnolia.contenttags.shortcut.TagShortcutAwareWorkbenchView</type>
<implementation>info.magnolia.contenttags.shortcut.TagShortcutAwareWorkbenchView</implementation>
</component>
</components>
There is plenty of <version> parameter usage in submodules poms, most of them are not needed because they are defined in dependencyManagement, but some of them should be moved dependencyManagement. |
| Comment by Antti Hietala [ 17/Oct/17 ] |
|
mdivilek, let's not bundle the magnolia-content-tags-personalization module in the 5.6 release. The Tags trait needs more work. Including it in the bundle without documentation or demo would be confusing. |
| Comment by Roman Kovařík [ 18/Oct/17 ] |
I'd remove whole p13n submodule from git then.
It has to find page variants with tags trait. But if we'll remove the p13n module as mentioned above, you don't need to thing about it...
Seems that you have an old demo (the UI tests for tagging are passing). |
| Comment by Christoph Meier [ 18/Oct/17 ] |
|
My point #2 above was wrong. |