Problem/Motivation

Prior to #3357741: Make config enforcement during cache-rebuild optional we did not have any settings for Config Enforce itself. As a result, it's essentially hidden in the "Configuration > Development" menu, rather than accessible via the tabs on the Enforced configs page.

Proposed resolution

Re-work the routing, etc.

User interface changes

  • Rename 'Settings' tab from `config_enforce_devel`
  • Add 'Settings' tab for `config_enforce`
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

ergonlogic created an issue. See original summary.

elber’s picture

Assigned: Unassigned » elber

I will work on it.

elber’s picture

Assigned: elber » Unassigned
Status: Active » Needs review

Hi I added my suggestion please revise.

tanuja bohra’s picture

StatusFileSize
new21.25 KB

Verified and tested the MR 24 , it's worked fine on 9.5.8. I'm suggesting adding an icon for this will be perfect.

elber’s picture

Status: Needs review » Reviewed & tested by the community

Hi moving to RTBC based in the previously comment.

star-szr’s picture

Assigned: Unassigned » star-szr
Status: Reviewed & tested by the community » Needs work

I'm taking another look at this, I don't think we want to create a new top-level menu item under Configuration.

star-szr’s picture

Version: 1.0.x-dev » 2.0.x-dev

star-szr changed the visibility of the branch 3360975-add-settings-tab to hidden.

star-szr’s picture

Assigned: star-szr » Unassigned
Status: Needs work » Needs review
StatusFileSize
new53.37 KB

New tab layout - Settings, Enforced configs, Enforcement settings

MR is ready for review/testing, and attaching a screenshot.

I renamed the existing Config Enforce Devel settings tab from Settings to Enforcement settings. The page title when on the CED settings page is Config Enforce Devel settings but this seemed way too long, and Devel settings didn't seem right either, seeming to refer instead to the Devel module.

  • star-szr committed 5a936ade on 2.0.x
    test: #3360975 Add and fix up config_enforce_devel Behat suite
    

  • star-szr committed e79f97fd on 2.0.x
    test: #3360975 Drop irrelevant test scenario
    
    This only makes sense to...
star-szr’s picture

I used the wrong issue # in those two commits, that was supposed to be #3570468: Fix up Behat suite.

star-szr’s picture

Reviewed with @ergonlogic and we agreed on some changes, merging shortly.

star-szr’s picture

Adding a related feature request issue that @ergonlogic and I agreed on while reviewing these changes.

  • star-szr committed 029f4b5a on 2.0.x
    test: #3360975 Revise test scenario for new tab name
    

  • star-szr committed d3b88a0c on 2.0.x
    feat: #3360975 Rework admin tabs
    
    - Add Settings tab (local task) for...
star-szr’s picture

Status: Needs review » Fixed

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.

Status: Fixed » Closed (fixed)

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