Problem/Motivation

If you install the module you can't find the Configure page from the module description.

Proposed resolution

Add the configure page to the .info.yml file

Read more at: Let Drupal 8 know about your module with an .info.yml file

Remaining tasks

None.

User interface changes

You will have the Configure link in the module section in the /admin/modules page

API changes

None.

Data model changes

None.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

adriancid created an issue. See original summary.

TravisCarden’s picture

That's odd. The configuration page is already specified in the .info.yml file as checklistapi.checklists.seo_checklist. That's definitely the correct route, and I'm sure it used to work. Maybe it doesn't work with dynamic routes and only used to work because we had to hardcode the route specification in the module due to an API bug for a while, and thus ceased working when we removed the hardcoded route. In any case, I'm afraid I don't have bandwidth to look into this, but if someone else figures it out I'll gladly commit a patch.

adriancid’s picture

Status: Active » Needs review
FileSize
304 bytes

@TravisCarden I think that is a core issue, I just create #2917439: info.yml file doesn't recognize dynamic routes meanwhile I think that we use the checklistapi.report route that refers to the admin/reports/checklistapi page.

If you have another idea I can made another patch.

tim.plunkett’s picture

While building the links for /admin/modules, access checking is run.

checklistapi.checklists.seo_checklist is the route name
It is set up by \Drupal\checklistapi\Routing\ChecklistapiRoutes::routes()
Which adds _checklistapi_access as an access requirement
Which corresponds to \Drupal\checklistapi\Access\ChecklistapiAccessCheck
Which denies access without a checklist_id request attribute.

tim.plunkett’s picture

adriancid’s picture

Status: Needs review » Closed (won't fix)