- ๐ฎ๐ณIndia hetal.solanki
Hetal.Solanki โ made their first commit to this issueโs fork.
- Status changed to Needs work
5 months ago 7:57am 18 July 2024 - First commit to issue fork.
- Merge request !2Issue #3305083:fixed Specify the route to the config form. โ (Open) created by Unnamed author
- Status changed to Needs review
5 months ago 8:17am 18 July 2024 - ๐ฎ๐ณIndia vinmayiswamy
Hi, I reviewed MR !2 on Drupal 10.2.x.
The MR applied cleanly.vinmayi.nidubrolu@********** ckeditor_exclude_tags % git apply -v 2.diff Checking patch ckeditor_exclude_tags.info.yml... Applied patch ckeditor_exclude_tags.info.yml cleanly. vinmayi.nidubrolu@********** ckeditor_exclude_tags % vinmayi.nidubrolu@********** ckeditor_exclude_tags % ddev drush cr [success] Cache rebuild complete. vinmayi.nidubrolu@********** ckeditor_exclude_tags % vinmayi.nidubrolu@********** ckeditor_exclude_tags % cat ckeditor_exclude_tags.info.yml name: 'Ckeditor Exclude tags' description: 'Exclude HTML tags in Ckeditor' type: module package: editor core_version_requirement: ^8 || ^9 || ^10 configure: ckeditor_exclude.admin_settings vinmayi.nidubrolu@********** ckeditor_exclude_tags %
Testing steps:โจ
1. Installed and enabled the "Ckeditor Exclude Tags" module version 1.0.x-dev.
2. Navigated to Admin > Extend.
3. Verified that the "Configure" route is accessible from the module's details expansion.Test Result:โจ
After applying the MR, the "Configure" route is now accessible from the module's details.Note: The configuration page currently displays as empty.
I've attached before and after screenshots for reference.
RTBC +1
Thanks!
- Status changed to RTBC
5 months ago 11:20am 18 July 2024 - ๐ฎ๐ณIndia vinmayiswamy
Correction needed: I mistakenly swapped the labels on the attached screenshots.
Here are the correct labels:
Before:
After:
Thanks!