Problem/Motivation

After a fresh installation of the standard Drupal profile, the Glossary views config has a dependency on system.main.menu has it provides a link into the main menu.

But this config dependency is not in the views.view.glossary.yml of the node module.

Proposed resolution

Update the views.view.glossary.yml file.

Remaining tasks

  • Provide a patch
  • Review it

User interface changes

None

API changes

None

Data model changes

None

CommentFileSizeAuthor
#2 drupal-glossary_config_update-2880142-2.patch802 bytesgrimreaper

Comments

Grimreaper created an issue. See original summary.

grimreaper’s picture

grimreaper
Primary language French
Location France 🇫🇷
commented
Assigned: grimreaper » Unassigned
Issue summary: View changes
Status: Active » Needs review
StatusFileSize
new drupal-glossary_config_update-2880142-2.patch802 bytes

Here is the patch.

dawehner’s picture

dawehner
Primary language German
commented

Is that really the only view which needs resaving? I think for making core committers happier resaving all would be helpful.

grimreaper’s picture

grimreaper
Primary language French
Location France 🇫🇷
commented
Status: Needs review » Closed (won't fix)

Hello,

Ok let's focus on the resaving all views.

I though by only touching one view it would be commited faster.

I have done a patch on #2303409: Resave all the views.