- 🇺🇸United States joshuami Portland, OR
+1 to adding some sort of notification to status report that would show for both Drupal 9.5 and Drupal 10 installations.
The change record by itself is not adequate to alert less experienced devs that they have a blocker to upgrading to Drupal 10 when using core's ckeditor module versus the contrib module.
My only recommendation would be to display the information as a warning rather than an error. I know that the clients I've helped establish CI/CD practice have tests that fail if "no errors found" is not in the status page. Could this be an error in Drupal 10 and a warning in Drupal 9? It would allow continued development not related to CKeditor and Core upgrades without breaking typical site-specific tests.
- 🇺🇸United States joshuami Portland, OR
Another thought that I touched on in Incorrect version of CKEditor being parsed from info.yml files 🐛 Incorrect version of CKEditor being parsed from info.yml files Active , it might be good to link to some documentation that suggests possible upgrade paths in the warning.
Resolving the Composer dependencies to get the contrib ckeditor module is place is not straightforward. At a minimum, a link to the change record → , the composer issue 🐛 Incorrect version of CKEditor being parsed from info.yml files Active , and the deprecated and obsolete extensions page → would be helpful.
From a release standpoint, it is even possible to get this sort of warning included in Drupal 9 at this point? Or is it only possible for us to alert the developers after they've tried to upgrade to 10 or run Upgrade Status → checks?
- 🇧🇪Belgium wim leers Ghent 🇧🇪🇪🇺
Sounds like this could have prevented 🐛 TypeError: array_intersect(): Argument #1 ($array) must be of type array, string given in array_intersect() (line 203 of .../core/modules/ckeditor5/src/Plugin/CKEditor5PluginManager.php). Postponed: needs info .
- 🇧🇪Belgium wim leers Ghent 🇧🇪🇪🇺
This would likely have prevented 💬 ckeditor plugin does not exist Fixed .
- Status changed to Closed: works as designed
12 months ago 10:23am 13 February 2024 - 🇧🇪Belgium wim leers Ghent 🇧🇪🇪🇺
I tried many times to push this forward but failed to convince core committers.
Closing this while triaging the issue queue.