Can't document menus

Created on 6 March 2025, 28 days ago

Problem/Motivation

This module has lost the ability to document menus. You can turn it on in the configuration, but when you go to add a CM Document for a menu none of the existing menus show up. The closest option is to document 'menu_link_content' but that is not really the same thing.

Steps to reproduce

  1. Go to admin /admin/config/system/cm_document and enable documenting menus
  2. Go try to create a content model document /admin/structure/cm_document/add
  3. Pull down the list for "Documentation for"
  4. Observe that any existing menus are not listed so they can not be documented.

Proposed resolution

I believe that at some point this was mistakenly changed to menu_link_content which is not the same as menus.

🐛 Bug report
Status

Active

Version

1.0

Component

Code

Created by

🇺🇸United States swirt Florida

Live updates comments and jobs are added and updated live.
Sign in to follow issues

Merge Requests

Comments & Activities

  • Issue created by @swirt
  • 🇮🇳India nidhi27

    Hi @swirt,

    How it was working earlier?

  • 🇺🇸United States swirt Florida

    How it should work is that if you have menus defined, you should be able to create a CM Document that can describe any menu.

    Example: If you had a "main" menu and an "FAQs" menu, you should be able to create a CM Document for the FAQ menu and then another CM Document for the main menu.

  • 🇮🇳India nidhi27

    Hi @swirt,

    Got it your explanation. I observed that if menus are fieldable at that time its showing in the option.

    For example, If you install menu_item_extras module at that time all the menu options are showing.

    Kindly confirm is it expected behaviour or not.

  • 🇺🇸United States swirt Florida

    Interesting discovery. No I think we want menus to be documentable whether they are fieldable or not. Lots of sites have menus they would want to document even if they were not fieldable.

  • Merge request !34Can't document menus resolved. → (Merged) created by nidhi27
  • 🇮🇳India nidhi27

    Hi @swirt,

    I have created MR #34 to resolve this issue. Let me know if there are any changes needed.
    Thanks!

  • 🇺🇸United States swirt Florida

    Thank you so much @Nidhi27. I will take a look at this a little later today.

  • 🇺🇸United States swirt Florida

    Thank you @nidhi27 for your work on this. It was very helpful.

Production build 0.71.5 2024