[MGNLDAM-940] Dam detail subApp should have presentable tab caption Created: 22/Feb/21 Updated: 26/Feb/21 Resolved: 25/Feb/21 |
|
| Status: | Closed |
| Project: | Magnolia DAM Module |
| Component/s: | None |
| Affects Version/s: | None |
| Fix Version/s: | 3.0.6 |
| Type: | Improvement | Priority: | Neutral |
| Reporter: | Milan Divilek | Assignee: | Milan Divilek |
| Resolution: | Fixed | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Attachments: |
|
||||||||
| Issue Links: |
|
||||||||
| Template: |
|
||||||||
| Acceptance criteria: |
Empty
|
||||||||
| Task DoD: |
[X]*
Doc/release notes changes? Comment present?
[X]*
Downstream builds green?
[X]*
Solution information and context easily available?
[X]*
Tests
[ ]*
FixVersion filled and not yet released
[ ] 
Architecture Decision Record (ADR)
|
||||||||
| Date of First Response: | |||||||||
| Epic Link: | UI framework implementation | ||||||||
| Sprint: | UI FW 22 | ||||||||
| Story Points: | 0 | ||||||||
| Description |
| Comments |
| Comment by Šimon Demočko [ 25/Feb/21 ] |
|
Hi mdivilek, I think Title would be more representable here than name. See Could we change it to use Title in this case? (edit) Okay, I found this comment. Still, could we override the default method so it would become possible to use Title and fallback to name when empty? |
| Comment by Milan Divilek [ 26/Feb/21 ] |
|
As discussed the title property doesn't have to be set, but the name property is automatically created when creating the asset and that's why we decided to stick with name property. |
| Comment by Šimon Demočko [ 26/Feb/21 ] |
|
We also considered implementing describeByProperty to accept an array of values from most specific to least specific, falling back to the next one, if the first one isn't available. This would require more changes and at this moment didn't seem like a worthwhile effort. |