Problem/Motivation

The configure link missing for the module on the "/admin/modules" page.

Steps to reproduce

Enable the "Automatic Updates" module.

Proposed resolution

Add configure link in the .info.yml file.

Remaining tasks

Adding code to the module's info.yml file. Either it should take to the "/admin/reports/updates/settings" page or "/admin/reports/updates" page.

Along with this the "Automatic Updates Extensions" should also have the configure link to its settings page which is available at "/admin/reports/updates/automatic-update-extensions"

User interface changes

NA

API changes

NA

Data model changes

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:

Comments

Prashant.c created an issue. See original summary.

prashant.c’s picture

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

sandipta’s picture

StatusFileSize
new403 bytes

Adding patch as per problem statement

prashant.c’s picture

Issue summary: View changes
sandipta’s picture

StatusFileSize
new1010 bytes

Adding Patch for additional requirement

phenaproxima’s picture

Status: Active » Needs work

Looks like this needs a reroll.

Gaurav Gupta changed the visibility of the branch 3406122-add-configure-link to active.

gaurav gupta’s picture

Status: Needs work » Needs review
StatusFileSize
new1002 bytes

Reroll patch for version 3.0.x

vivek panicker’s picture

Code looks good.
Also requirements are fulfilled.
Hence moving to RTBC.

vivek panicker’s picture

Status: Needs review » Reviewed & tested by the community
phenaproxima’s picture

Status: Reviewed & tested by the community » Needs review

I'm not sure this patch makes sense as-is. The configure link for Automatic Updates Extensions isn't really a configuration page, so...why is it there? Can you explain the rationale for pointing it to that specific route?

gaurav gupta’s picture

@phenaproxima
Yes, I agree since Automatic Updates Extensions does not have any configuration form we can skip this.
Made the changes.
Thanks

phenaproxima’s picture

Status: Needs review » Needs work

Tests appear to be failing...?

gaurav gupta’s picture

@phenaproxima
I am not sure if i have made changes that made pipeline failed.
All i have done is solve conflicts and revert some changes.
Thanks.

anybody’s picture

Version: 3.0.0-beta1 » 3.1.x-dev

Tests are failing because this targets the wrong branch. The fork needs to be updated so that the 3.1.x branch becomes available as merge target.

Gaurav Gupta changed the visibility of the branch 3406122-add-configure-link to hidden.

Gaurav Gupta changed the visibility of the branch automatic_updates-3406122-3406122-configure-link to hidden.

prashant.c’s picture

Rebased with 3.1.xLets see the tests pass or not.

prashant.c’s picture

Prashant.c changed the visibility of the branch 3406122-configure to hidden.

prashant.c’s picture

Status: Needs work » Needs review

There were a lot of out-of-scope changes in the previously raised MRs, raised a new one, and made others hidden. Once the tests pass we can change the issue status to NR

prashant.c’s picture

Status: Needs review » Needs work
gaurav gupta’s picture

Status: Needs work » Needs review
tirupati_singh’s picture

Status: Needs review » Reviewed & tested by the community
StatusFileSize
new22.47 KB
new24.35 KB

Hi, I've applied the provided MR as a patch and it applied cleanly with no errors. Before applying patch there was no Configuration link for the module on the Extend page /admin/modules. On applying the patch, module configuration link is now added on the Extend page and admin/reports/updates/settings page opens when clicking on the Configure link. I've attached the before and after screenshots for reference.

Thanks!

sourav_paul’s picture

Yeah, MR !1090 works for me as well.
It attached a configure link to the module on the extend page.

RTBC+1

phenaproxima’s picture

phenaproxima’s picture

Title: Add Configure link to Module. » Add Configure link to info file

phenaproxima’s picture

Status: Reviewed & tested by the community » Fixed

Merged into 3.1.x, thanks!

Status: Fixed » Closed (fixed)

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