- Issue created by @s_leu
- Merge request !13Issue #3280990 by DuaelFr: test that the fields are available according to the... โ (Open) created by s_leu
- Status changed to Needs review
almost 2 years ago 5:00am 3 October 2023 - Status changed to Needs work
almost 2 years ago 12:54pm 10 October 2023 - ๐บ๐ฆUkraine vselivanov Kyiv, Ukraine
Hi @s_leu!
Thank you for work with this warnings.But we should create a merge request to 2.x branch - not 8.x-1.x.
So we need only this commitOnly from this commit I created a patch that works good with the latest Drupal 10.1.5, CKEditor5 and Editor Advanced Link 2.2.4 release.
- Status changed to Needs review
over 1 year ago 2:28pm 13 December 2023 - Status changed to RTBC
25 days ago 1:22pm 8 July 2025 - ๐ฆ๐บAustralia jannakha Brisbane!
thanks for your work!
tested on D11 - no more warnings! - ๐ณ๐ฑNetherlands jeroen dost
Hi @vselivanov,
The patch does not work for the latest release of this module (v2.3.1). Can you provide an updated version? Or should I downgrade (using core v10.5.1).
The issue still exists. Does this warning prevent it from working correct, or is it just a deprecation warning?Thanks
- ๐ฆ๐บAustralia jannakha Brisbane!
@jeroen dost - try patch from https://www.drupal.org/project/editor_advanced_link/issues/3534699 ๐ Refactor custom JS for CKEditor5 v45+ Active for 2.3
- ๐บ๐ธUnited States jcandan
My bad--I didn't explain why I'd switched the version. It seems
2.x-dev
should be deleted while a minor dev releases exists. See ๐ Fix Composer stable, caret constraintย installs dev release Active .@jeroen dost, I agree this affects both, but it seems--as with many other issues-- ๐ Refactor custom JS for CKEditor5 v45+ Active will address this issue. I wonder if we need to ensure the scope of that ticket includes this, or just mark this as a
2.x-dev
issue.I can see an argument for either way. Just having this note here may be sufficient. :)