-
Notifications
You must be signed in to change notification settings - Fork 332
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[FEATURE] Added navigation context in structure #182
Conversation
@@ -640,7 +661,7 @@ public function loadStartPage($webspaceKey, $languageCode) | |||
$startPage = $this->load($uuid, $webspaceKey, $languageCode); | |||
$startPage->setNodeState(StructureInterface::STATE_PUBLISHED); | |||
$startPage->setGlobalState(StructureInterface::STATE_PUBLISHED); | |||
$startPage->setNavigation(true); | |||
$startPage->setNavContexts(true); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@wachterjohannes Should here really be true
passed? NavContexts
are a string array.
Generally ok, but the navContexts variable confuses me a bit... What does it mean when it is a boolean? Shouldn't it always be a string array, with the option to be empty, if it's not shown in any navigation? |
…f/sulu into feature/navigation-context Conflicts: src/Sulu/Component/Content/Mapper/ContentMapper.php
* Add AudienceTargetingBundle Javascript * Remove registration of AudienceTargetingBundle * Add database as a service
Added navigation context in structure:
Tasks:
Informations: