Keep MenuTreeStorage field definition in sync with MenuLinkManager

Comments & Activities

Not all content is available!

It's likely this issue predates Contrib.social: some issue and comment data are missing.

Comment over 1 year ago →
🇺🇸United States smustgrave
This issue is being reviewed by the kind folks in Slack, #needs-review-queue-initiative. We are working to keep the size of Needs Review queue [2700+ issues] to around 400 (1 month or less), following Review a patch or merge request → as a guide.

Tagged for novice for beginner developers to reroll.
Comment over 1 year ago →
RISHABH VISHWAKARMA
Adding patch for 10.1.x-dev. Please review.
Comment over 1 year ago →
🇧🇪Belgium borisson_ Mechelen, 🇧🇪
The phpcs failure in #22 needs to be fixed.
Comment over 1 year ago →
🇮🇳India pooja saraah Chennai
Fixed failed commands on #22
Attached patch against Drupal 10.1.x
Attached interdiff
Status changed to Needs review over 1 year ago6:07pm 27 January 2023
Comment over 1 year ago →
🇺🇾Uruguay rpayanm
Status changed to RTBC over 1 year ago8:43pm 27 January 2023
Comment over 1 year ago →
🇺🇸United States smustgrave
Reroll looks good.

Good job @pooja saraah!

catch → committed 475a3c73 on 10.1.x

Issue #2302085 by voleger, pooja saraah, Rishabh Vishwakarma, victoru:...

Status changed to Fixed over 1 year ago2:07pm 21 February 2023
Comment over 1 year ago →
🇬🇧United Kingdom catch
Looks great. Don't think we need a CR for this trait, it's quite internal but good to share the code between the two places in core.

Committed 475a3c7 and pushed to 10.1.x. Thanks!
Comment over 1 year ago →
🇬🇧United Kingdom catch
Comment over 1 year ago →
System Message
Automatically closed - issue fixed for 2 weeks with no activity.

Comments & Activities