- Issue created by @dieterholvoet
- π§πͺBelgium dieterholvoet Brussels
The minimum Drupal version is bumped to 9.2 because of the usage of the usage of the new field widget hook β . I also added back the Image module dependency since it was incorrectly removed in π¬ Support Drupal 10 Fixed . Setting to Needs work for the upgrade path implementation.
- Merge request !12Resolve #3472997 "Rewrite module to use default field storage" β (Open) created by dieterholvoet
- Status changed to Needs work
7 months ago 9:33am 9 September 2024 - π«π·France bachbach Le Havre
It's reinventing the wheel, since Drupal has a whole field storage system built in that automatically deals with things like translations and revisions.
exactly what i though after hours trying to make caption in nested parragraph translation to work :(
then i found your issue and your patch work like a charm!had to :
pmu the module (obviously delete all the caption contents)
composer require dev version of module
apply your patch
enable the module againthanks for your work !
- Assigned to Foxy-vikvik
- π§πͺBelgium dieterholvoet Brussels
@foxy-vik why did you make the caption a plain text field instead of a formatted text field in #e2e9925? In the current version of the module it's also a formatted text field, so I don't see why you would change that.
- πΊπ¦Ukraine Foxy-vikvik
@dieterholvoet It was a mistake. I found an issue and changed it. This merge request is still a work in progress. So it will be formatted text. I will change it when I have time.
- First commit to issue fork.