Skip to content
This repository was archived by the owner on Sep 16, 2021. It is now read-only.

Fix menu elements root #127

Merged
merged 1 commit into from
Dec 12, 2017
Merged

Conversation

EmmanuelVella
Copy link
Contributor

Q A
Branch? master
Bug fix? yes
New feature? no
BC breaks? no
Deprecations? no

Hello !

When editing a menu, the elements (children) root should be the menu path (for example /cms/menu/top), not /cms/menu. Right ?

dbu
dbu previously approved these changes Dec 12, 2017
Copy link
Member

@dbu dbu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks correct to me. i fixed the styleci on master, the build fail seems unrelated to this change.

@EmmanuelVella
Copy link
Contributor Author

Thank you David ! I just rebased my branch.

Copy link
Member

@dbu dbu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thank you!

@dbu dbu merged commit 6dc1584 into symfony-cmf:master Dec 12, 2017
@EmmanuelVella EmmanuelVella deleted the elements-root branch December 12, 2017 16:23
ElectricMaxxx pushed a commit that referenced this pull request Jan 12, 2018
Improvements to run the tests

use new testing version

remove deprecated method

DevKit updates

DevKit updates

DevKit updates

DevKit updates

DevKit updates

move files to new test app structure

DevKit updates

new test app structure

fixes due to StyleCI

fix phpcr-odm version

make all admin services public

make a block admin public too

add admin bundles to own bundles.php

DevKit updates

DevKit updates

DevKit updates

DevKit updates

DevKit updates

DevKit updates

DevKit updates

DevKit updates

DevKit updates

DevKit updates

DevKit updates

DevKit updates

DevKit updates

do not require non existing bundle set anymore

remove fos jsrouting-bundle decleration
ElectricMaxxx added a commit that referenced this pull request Jan 23, 2018
…nts on way to SF4 compatiblility

* Fix menu elements root (#127)

Improvements to run the tests

use new testing version

remove deprecated method

DevKit updates

DevKit updates

DevKit updates

DevKit updates

DevKit updates

move files to new test app structure

DevKit updates

new test app structure

fixes due to StyleCI

fix phpcr-odm version

make all admin services public

make a block admin public too

add admin bundles to own bundles.php

DevKit updates

DevKit updates

DevKit updates

DevKit updates

DevKit updates

DevKit updates

DevKit updates

DevKit updates

DevKit updates

DevKit updates

DevKit updates

DevKit updates

DevKit updates

do not require non existing bundle set anymore

remove fos jsrouting-bundle decleration

* remove use stmt.

* use newer menu-bundle version

* use a fork for symfony compat of the ckeditor bundle

* fix composer.json

* use own branch

* avoid private service deprecation by using new compile pass in testing component

* fix repository

* use other repository url

* revert ckeditor change

* remove symfony dependency
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants