Replace twitter, video and image processors

Created on 20 May 2024, 27 days ago
Updated 7 June 2024, 9 days ago

Split off from parent to keep some clarity / a reviewable chunk.

These 3 processors all need a new 'flattened' option in the entityreference formatter.

Gallery not included yet, because that requires extra work.

πŸ“Œ Task
Status

Fixed

Version

3.0

Component

Code

Created by

πŸ‡³πŸ‡±Netherlands roderik Amsterdam,NL / Budapest,HU

Live updates comments and jobs are added and updated live.
Sign in to follow issues

Merge Requests

Comments & Activities

  • Issue created by @roderik
  • Merge request !65Convert twitter, video and image paragraphs. β†’ (Merged) created by roderik
  • Status changed to Needs review 27 days ago
  • πŸ‡³πŸ‡±Netherlands roderik Amsterdam,NL / Budapest,HU
  • Status changed to Needs work 26 days ago
  • πŸ‡¦πŸ‡Ή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
  • πŸ‡³πŸ‡±Netherlands roderik Amsterdam,NL / Budapest,HU
  • Status changed to Needs work 23 days ago
  • πŸ‡³πŸ‡±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
  • πŸ‡³πŸ‡±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.

  • Pipeline finished with Skipped
    23 days ago
    #181381
    • roderik β†’ committed 59fd97f7 on 3.x
      Issue #3448384 by roderik, fago: Replace twitter, video and image...
  • Status changed to Fixed 23 days ago
  • πŸ‡³πŸ‡±Netherlands roderik Amsterdam,NL / Budapest,HU
  • Automatically closed - issue fixed for 2 weeks with no activity.

Production build 0.69.0 2024