Updated NodeRouteContext to work with layout builder

Created on 11 December 2019, over 5 years ago
Updated 9 May 2023, about 2 years ago

Context should be updated so it is capable of responding on layout-builder builder routes.

A similar exception should be devised to how it currently responds on node-add routes.

Feature request
Status

Active

Version

10.1

Component
Node system 

Last updated 1 day ago

No maintainer
Created by

🇦🇺Australia dpi Perth, Australia

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

Comments & Activities

Not all content is available!

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

  • 🇦🇺Australia dpi Perth, Australia
  • Status changed to Postponed: needs info 4 months ago
  • 🇦🇺Australia acbramley

    Just tested this and it works for routes provided by OverridesSectionStorage (i.e /node/1/layout) but not DefaultsSectionStorage (i.e /admin/structure/types/manage/article/display/default/layout)

    Was this ticket referring to the latter?

  • Status changed to Closed: outdated 2 days ago
  • 🇦🇺Australia dpi Perth, Australia

    Perhaps the issue was implicitly resolved by #3099968: Layout builder incorrectly resolves global contexts values when viewing layouts or another intermediate issue.

    I can see node context is resolving on /layout pages, I dont recall if there was a particular way/order-of-operations that made this not work.

    I'll close this one out.

    Anyone can feel free to add better steps to reproduce in case there are more awkward ways of replicating the bug.

Production build 0.71.5 2024