- 🇺🇸United States bluegeek9
I think this is worth adding. Other entities that are not revisioned still have a changed field, like comments.
- Assigned to Tyapchyc
- Merge request !39Issue #3192887: Add a changed timestamp field to message entities → (Merged) created by Tyapchyc
- Issue was unassigned.
- Status changed to Needs review
5 months ago 8:01am 17 June 2024 - Status changed to Needs work
5 months ago 8:25pm 17 June 2024 - 🇺🇸United States loze Los Angeles
+1 for adding this.
The MR does add the field successfully, however I think we need to add a few things.
1. Add an update hook that copies all of the existing message entities created timestamps to the new changed field. Currently they are all null.
2. Editing a message entity on the entity edit pagemessage/%/edit
does not update the changed date. - Assigned to Tyapchyc
- Issue was unassigned.
- Status changed to Needs review
5 months ago 12:49pm 21 June 2024 - 🇺🇸United States loze Los Angeles
Thanks, The update hook worked for me.
My commit gets the changed timestamp to update when the entity is saved. It appears to be working as expected in my local environment. - Status changed to Fixed
3 months ago 11:01am 26 August 2024 -
amitaibu →
committed be262b72 on 8.x-1.x authored by
tyapchyc →
Issue #3192887: Add a changed timestamp field to message entities
-
amitaibu →
committed be262b72 on 8.x-1.x authored by
tyapchyc →
Automatically closed - issue fixed for 2 weeks with no activity.