Skip to content
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

Implemented page-mover command #206

Merged
merged 2 commits into from
Jun 28, 2017

Conversation

wachterjohannes
Copy link
Member

@wachterjohannes wachterjohannes commented Jun 23, 2017

Q A
Bug fix? no
New feature? yes
BC breaks? no
Deprecations? no
Fixed tickets none
Related issues/PRs built ontop of #208
License MIT

What's in this PR?

This PR adds a command to move articles from one parent-page to another one.

Why?

If you remove a page in the page tree. The articles will be detached from the page (URL stays the same) and you can choose a new page when you edit the Article. But if you want to do this for all the affected Articles there is no way to do it in a single step.

This PR adds the command and the functionality todo that. Later this functionality can be triggered by the content-manager from the UI (additional later PR when needed).

Example Usage

bin/console sulu:article:page-tree:move /page-1 /page-2 sulu_io de

To Do

  • Finish prototype
  • Tests
  • Add documentation page

@wachterjohannes wachterjohannes force-pushed the feature/page-mover branch 7 times, most recently from fa5f580 to 0bbc12a Compare June 26, 2017 07:20
@wachterjohannes wachterjohannes changed the title Implemented page-mover prototype Implemented page-mover command Jun 28, 2017
@wachterjohannes wachterjohannes added this to the 0.6.0 milestone Jun 28, 2017
@alexander-schranz alexander-schranz merged commit a60ac4f into sulu:develop Jun 28, 2017
@wachterjohannes wachterjohannes deleted the feature/page-mover branch June 29, 2017 04:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants