- Issue created by @bwaindwain
- Status changed to Needs review
5 months ago 9:58pm 25 June 2024 - Status changed to RTBC
5 months ago 8:57am 26 June 2024 - 🇮🇳India Sandeep_k New Delhi
I've tested shared MR- MR !7 mergeable on Drupal version- 10.3.0-dev. The patch was applied successfully and looks good to me.
Testing Steps:
- Add/download Drupal 10.3.0
- Add/download Module "CKEditor Find/Replace".
- Go to /admin/config/content/formats- and add Find/Replace to CK Editor.
- Go to Content/select text format with updated editor. Before patch error attached.
- Download this share MR and Apply.
- Go back to the Content and Re-verify this
Testing Results:
After applying the patch, the TypeError is resolved, added after result. RTBC++ - Status changed to Postponed: needs info
5 months ago 10:37am 26 June 2024 - 🇺🇸United States fathershawn New York
This is an opportune time to evaluate retiring this module. The team at CKSource (developers of CKEditor) have released CKEditor 5 Plugin Pack → which includes the plugin provided by this module.
Please evaluate their officially supported module and add a note back if
ckeditor_find
is still needed. - Status changed to Closed: won't fix
5 months ago 3:45pm 1 July 2024 The plugin pack works. Thanks.
composer require 'drupal/ckeditor5_plugin_pack:^1.1' drush en ckeditor5_plugin_pack ckeditor5_plugin_pack_find_and_replace drusn pmu ckeditor_find drush cex -y
I'd suggest marking the module as deprecated if you don't intend to support it alongside the plugin pack, and put a link/reference to the alternative on the module page.
- 🇺🇸United States fathershawn New York
Yes, that's the plan. I needed to test or someone to test the replacement first so thanks for the report
- 🇺🇸United States jrockowitz Brooklyn, NY
I can confirm that the above steps work as expected
composer require 'drupal/ckeditor5_plugin_pack:^1.1' drush en ckeditor5_plugin_pack ckeditor5_plugin_pack_find_and_replace drusn pmu ckeditor_find drush cex -y
- 🇫🇮Finland Alexander Tallqvist
I can also confirm that this does the trick. No need to configure any settings for the text format.
composer require 'drupal/ckeditor5_plugin_pack:^1.1' drush en ckeditor5_plugin_pack ckeditor5_plugin_pack_find_and_replace drush pmu ckeditor_find drush cex -y
- 🇨🇦Canada smulvih2 Canada 🍁
#10/#11 work for me. I would also add a
composer remove drupal/ckeditor_find
to remove ckeditor_find from the project. @smulvih You can't do that in the same step if you have CI/CD, because you can't uninstall the module when the code has been removed from the repository.
- First commit to issue fork.