If a module offers a configuration form, then we can specify the route to this form in the info file. It will then show up as a link in the Extend page (at /admin/modules URL path) when the user expands the details.

CommentFileSizeAuthor
#9 config-page.png123.11 KBvinmayiswamy
#9 extend-page-after-MR.png291.09 KBvinmayiswamy
#9 extend-page-before-MR.png300.21 KBvinmayiswamy
#4 Screenshot from 2024-07-18 13-26-25.png148.3 KBhetal.solanki
#2 3160485-2.patch318 bytesgaurav_drupal

Issue fork ckeditor_exclude_tags-3160485

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks

Comments

Gaurav_drupal created an issue. See original summary.

gaurav_drupal’s picture

gaurav_drupal
he/him
Primary language Hindi
Location Indore
commented
Assigned: gaurav_drupal » Unassigned
Status: Needs work » Needs review
StatusFileSize
new 3160485-2.patch318 bytes

Provided the patch. Please review.

Hetal.Solanki made their first commit to this issue’s fork.

hetal.solanki’s picture

hetal.solanki
Primary language English
commented
StatusFileSize
new Screenshot from 2024-07-18 13-26-25.png148.3 KB

patch is not apply.

hetal.solanki’s picture

hetal.solanki
Primary language English
commented
Status: Needs review » Needs work

atul_ghate made their first commit to this issue’s fork.

atul_ghate opened merge request !2

atul_ghate’s picture

atul_ghate commented
Status: Needs work » Needs review
vinmayiswamy’s picture

vinmayiswamy commented
StatusFileSize
new extend-page-before-MR.png300.21 KB
new extend-page-after-MR.png291.09 KB
new config-page.png123.11 KB

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!

hetal.solanki’s picture

hetal.solanki
Primary language English
commented
Status: Needs review » Reviewed & tested by the community
vinmayiswamy’s picture

vinmayiswamy commented

Correction needed: I mistakenly swapped the labels on the attached screenshots.

Here are the correct labels:

Before:
before

After:
after

Thanks!