[MGNLDAMEXT-22] Add column showing last update date in cumulus asset app Created: 22/Oct/14  Updated: 24/Oct/14  Resolved: 24/Oct/14

Status: Closed
Project: DAM Extensions
Component/s: None
Affects Version/s: None
Fix Version/s: 1.0

Type: Improvement Priority: Neutral
Reporter: Jan Haderka Assignee: Evzen Fochr
Resolution: Fixed Votes: 0
Labels: None
Remaining Estimate: Not Specified
Time Spent: Not Specified
Original Estimate: Not Specified

Issue Links:
relation
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)
Date of First Response:

 Comments   
Comment by Jan Haderka [ 23/Oct/14 ]
  • What's the point of making last mod date field configurable when it's anyway retrieved from JSON object so it will be always valid for cumulus only?
  • If it stays configurable, please change name of the property to lastModificationFieldName.
  • And why are we hardcoding field uuid in the configuration again? This already exists in cumulus config so we should read it from there rather then setting same magic number elsewhere again ... the fewer places are exposed to cumulus internal ways of working the better.
Comment by Jan Haderka [ 23/Oct/14 ]

Cool, so now it's using asset prop only and there's nothing specific to cumulus, right? So why does it still stays in Cumulus then instead of being in damext?

Comment by Evzen Fochr [ 24/Oct/14 ]

Only last commit will be integrated (changes in dam-ext).

Generated at Mon Feb 12 00:40:50 CET 2024 using Jira 9.4.2#940002-sha1:46d1a51de284217efdcb32434eab47a99af2938b.