- Issue created by @roderik
- Status changed to Needs review
27 days ago 10:34pm 20 May 2024 - Status changed to Needs work
26 days ago 11:53am 21 May 2024 - π¦πΉAustria fago Vienna
Commented, please see remarks above.
Generally, change see to go into the right direction, but I dislike including the improvement of our ER-formatter here.
This should gets its own dedicated issue, so it appears properly in release notes. When doing so, it gets its own description why it is needed, and should come with some quick kernel test to demonstrate how it works correctly from a simple isolated API perspective. Maybe best, let's still have this issue, do it first, and then get back to finishing up here? - Status changed to Needs review
24 days ago 9:50am 23 May 2024 - Status changed to Needs work
23 days ago 8:37am 24 May 2024 - π³π±Netherlands roderik Amsterdam,NL / Budapest,HU
per last remark.
I pretty much got ( / extracted) the OK to merge this when it's done, though.
- Status changed to Needs review
23 days ago 10:27am 24 May 2024 - π³π±Netherlands roderik Amsterdam,NL / Budapest,HU
I'll merge this as soon as i have the work on π Convert gallery processor Needs work done (so that MR won't be 'polluted' with this one).
Any remarks on the EntityReferenceCeFieldFormatter / VideoEmbedCeFieldFormatter I'll still happily address in a followup.
-
roderik β
committed 59fd97f7 on 3.x
Issue #3448384 by roderik, fago: Replace twitter, video and image...
-
roderik β
committed 59fd97f7 on 3.x
- Status changed to Fixed
23 days ago 2:39pm 24 May 2024 Automatically closed - issue fixed for 2 weeks with no activity.