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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

add parent when duplicating assembly #4371

Merged
merged 4 commits into from
Oct 25, 2018
Merged

Conversation

isaacmg410
Copy link
Contributor

@isaacmg410 isaacmg410 commented Oct 24, 2018

馃帺 What? Why?

When duplicate a children assembly it doesn't display in children assemblies backoffice. It's like you duplicate a mother assembly.

馃搶 Related Issues

馃搵 Subtasks

  • Add CHANGELOG entry
  • Add parent in CopyAssembly Command
  • Add tests

馃摲 Screenshots (optional)

Description

@ghost ghost assigned isaacmg410 Oct 24, 2018
@ghost ghost added the status: WIP label Oct 24, 2018
@@ -50,6 +50,7 @@ def copy_assembly
banner_image: @assembly.banner_image,
promoted: @assembly.promoted,
scope: @assembly.scope,
parent: @assembly.parent,
Copy link
Contributor

Choose a reason for hiding this comment

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

Maybe add tests for this?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

tests added

@isaacmg410 isaacmg410 force-pushed the fix_duplicate_children_as_mother branch from 3d8587a to ec2a711 Compare October 25, 2018 08:06
mrcasals
mrcasals previously approved these changes Oct 25, 2018
@mrcasals
Copy link
Contributor

mrcasals commented Oct 25, 2018

@isaacmg410 There are some lint errors!

@isaacmg410
Copy link
Contributor Author

@mrcasals now all is green 馃槃

@mrcasals mrcasals merged commit 48a435d into master Oct 25, 2018
@mrcasals mrcasals deleted the fix_duplicate_children_as_mother branch October 25, 2018 15:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants