Problem/Motivation

Currently the Features module shows a Configure link if Features UI is enabled. However, since the corresponding route is provided by Features UI, the link would more appropriately show for Features UI (parallel with, for example, Views and Views UI).

Proposed resolution

Move this line from features.info.yml to features_ui.info.yml:

configure: features.assignment

Remaining tasks

User interface changes

API changes

Data model changes

CommentFileSizeAuthor
#2 features-configure_link_in_features_ui-2635952-2.patch1.14 KBGrimreaper
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

nedjo created an issue. See original summary.

Grimreaper’s picture

Grimreaper
Primary language French
Location France 🇫🇷
CreditAttribution: Grimreaper as a volunteer commented
Status: Active » Needs review
FileSize
features-configure_link_in_features_ui-2635952-2.patch1.14 KB

Hello,

Here is a patch to fix this issue. If it helps you :).

Thanks for the review.

nedjo’s picture

nedjo
he/him/his
Primary language English
CreditAttribution: nedjo at Chocolate Lily commented
Status: Needs review » Reviewed & tested by the community

Looks good.

  • nedjo committed c3f8fee on 8.x-3.x authored by Grimreaper 
    Issue #2635952 by Grimreaper: Configure link for Features should be for...
nedjo’s picture

nedjo
he/him/his
Primary language English
CreditAttribution: nedjo at Chocolate Lily commented
Status: Reviewed & tested by the community » Fixed

Committed, thanks!

Grimreaper’s picture

Grimreaper
Primary language French
Location France 🇫🇷
CreditAttribution: Grimreaper as a volunteer commented

Thanks nedjo

Status: Fixed » Closed (fixed)

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