Problem/Motivation

The module provides a settings form route at `mixpanel.settings_form`, but `mixpanel.info.yml` does not declare a `configure` key. As a result, Drupal’s Extend page does not show a Configure link for the module even though a configuration form exists.

Steps to reproduce

  1. On Drupal 11.x, install Mixpanel.
  2. Go to `/admin/modules`.
  3. Find the Mixpanel module row.

Proposed resolution

Add `configure: mixpanel.settings_form` to `mixpanel.info.yml`.

Issue fork mixpanel-3605624

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

dkmishra created an issue. See original summary.

dkmishra opened merge request !10

dkmishra’s picture

dkmishra commented
Priority: Normal » Minor
dkmishra’s picture

dkmishra commented
Status: Active » Needs review
darvanen’s picture

darvanen
he/him
Primary language English
Location Sydney, Australia
commented
Status: Needs review » Needs work

This is the second tiny improvement you have made in a few days, if you have more up your sleeve add them to this issue please, I won't be crediting a third one like this :)

dkmishra’s picture

dkmishra commented

Hi @darvanen, thanks for the feedback. I'll keep that in mind and will try to group similar minor improvements together where possible rather than opening separate issues for each one.

I don't have any additional improvements for this project at the moment, but I'll be sure to add them to an existing issue if I find more related items in the future.

Thanks for reviewing and maintaining the project.

  • darvanen committed 262434eb on 2.x authored by dkmishra 
    fix: #3605624 Module info file is missing configure route

By: dkmishra...
darvanen’s picture

darvanen
he/him
Primary language English
Location Sydney, Australia
commented
Status: Needs work » Fixed

I don't so much want them added to existing issues as grouped together when they're one-liners like this.

Thanks for the update. Committed.

Now that this issue is closed, review the contribution record.

As a contributor, attribute any organization that helped you, or if you volunteered your own time.

Maintainers, credit people who helped resolve this issue.

dkmishra’s picture

dkmishra commented

Thanks for the clarification. I'll group similar minor improvements together in future issues where appropriate.

Thanks for the review and for committing the fix.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.