Since he is one of the original authors of the storage design for menu links and he's back in core development (yay!), it seems only natural to do this :)

CommentFileSizeAuthor
new_menu_link_maintainer.patch482 bytesamateescu

Comments

dawehner’s picture

dawehner
Primary language German
commented
Status: Needs review » Reviewed & tested by the community

+1 even his contributions are more in Drupal\Core instead of menu_link.module folder.

amateescu’s picture

amateescu commented

The menu_link module is mostly code that was moved from includes\menu.inc and turned to OO :)

dawehner’s picture

dawehner
Primary language German
commented

Maybe we should have a menu link component which deals with all this parts: local tasks/ local actions / contextual links etc.

catch’s picture

catch
he/him
Primary language English
commented
Assigned: Unassigned » pwolanin
Status: Reviewed & tested by the community » Needs review

Let's ask him directly first if he wants to be listed.

pwolanin’s picture

pwolanin commented

Yes, I'm more involved in the menu system issues, but there is significant overlap with this module too and I'll help clean it up as we go forward.

pwolanin’s picture

pwolanin commented
Status: Needs review » Reviewed & tested by the community
amateescu’s picture

amateescu commented
Assigned: pwolanin » Unassigned

Thanks, Peter.

alexpott’s picture

alexpott
he/they
Primary language English
Location 🇪🇺🌍
commented
Status: Reviewed & tested by the community » Fixed

Committed d8b5e11 and pushed to 8.x. Thanks!

Status: Fixed » Closed (fixed)

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