- Issue created by @vangelisp
- Status changed to Needs work
about 1 year ago 10:57am 10 October 2023 - 🇬🇧United Kingdom vangelisp Fife, Scotland
@nagy.balint if you think that this can work for the module, I can also provide an upgrade hook.
Additionally, I'm working on a revised view (I saw that you've merged the submodule/view) that will use the `diff` class to show the differences and also display the entry date.
- 🇭🇺Hungary nagy.balint
Hi!
I think it is a good idea to include that.
I wonder how the diff will look in that view.
- 🇬🇧United Kingdom vangelisp Fife, Scotland
Attaching a screenshot on how it looks.
I've actually hidden the data fields (excluded in the view) to make the diff appear properly
- 🇭🇺Hungary nagy.balint
Looks nice!
Not sure if we need some basic date filter.
- 🇬🇧United Kingdom vangelisp Fife, Scotland
I'm posting an updated .patch file that also includes the upgrade for the new column `originalData`.
This patch applies also some minor coding standards
- 🇭🇺Hungary nagy.balint
I think we need a new update hook, as otherwise sites that already run this update hook won't run this new part.
- 🇬🇧United Kingdom vangelisp Fife, Scotland
You're right!
Attaching a revised version of the latest patch with a separate upgrade hook (8002)
- last update
about 1 year ago 7 pass, 1 fail - last update
about 1 year ago 7 pass, 1 fail - Status changed to Needs review
about 1 year ago 1:12pm 11 October 2023 - last update
about 1 year ago 10 pass -
nagy.balint →
committed 9d82af8a on 3.0.x authored by
VangelisP →
Issue #3392837 by VangelisP, nagy.balint: Record new data...
-
nagy.balint →
committed 9d82af8a on 3.0.x authored by
VangelisP →
- Status changed to Fixed
about 1 year ago 1:47pm 11 October 2023 Automatically closed - issue fixed for 2 weeks with no activity.