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.

Sign up for GitHub

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

Jump to bottom

Make duplicating Context Resources optional when duplicating Context #13277

Closed
jaygilmore opened this issue Feb 6, 2017 · 2 comments
Closed

Make duplicating Context Resources optional when duplicating Context #13277

jaygilmore opened this issue Feb 6, 2017 · 2 comments
Labels
proposal Proposal about improvement aka RFC. Need to be discussed before start implementation. type-frontend Issues related to UI/UX issues, mostly about styles and frontend implementations on JavaScript.
Milestone
v2.6.0

Comments

@jaygilmore
Copy link
Member

jaygilmore commented Feb 6, 2017 •
edited
Loading

Summary

It should be optional to duplicate the Resources under a Context when duplicating the Context either from the tree or as suggested in #13276. Add checkbox to the dialog to confirm choice to duplicate Resources.

Step to reproduce

Trigger duplicating a Context from the Resource Tree. Dialog pops up with options. It should be another unchecked checkbox on the dialog box to confirm that they wish to duplicate the Resources as well.

Observed behavior

Currently, it is not possible to optionally duplicate Resources when duplicating a Context. The Resources will be Duplicated.

Expected behavior

duplicate_child_resources
Allow the user to choose to duplicate or not duplicate the children Resources of a Context. Add the checkbox input with the text: "Duplicate child Resources". It may also make sense to disable the Preserve duplicated resources alias and Preserve duplicated resources menu index checkboxes until the Duplicate child Resources has been checked.

Environment

Current MODX Revolution Managers +
@Jako Jako added type-frontend Issues related to UI/UX issues, mostly about styles and frontend implementations on JavaScript. proposal Proposal about improvement aka RFC. Need to be discussed before start implementation. priority-2-high labels Feb 6, 2017
@Jako Jako added this to the v2.6.0 milestone Feb 6, 2017
@joeke
Copy link
Contributor

joeke commented Mar 3, 2017

I'll look into this. #modxbughunt

@joeke joeke mentioned this issue Mar 3, 2017
Merged
@OptimusCrime OptimusCrime mentioned this issue Mar 3, 2017
Closed
Mark-H added a commit that referenced this issue Jun 15, 2017
@Mark-H
Merge remote-tracking branch 'upstream/pr/13321' into 2.x
d42be44 
* upstream/pr/13321:
  Fixed issue #13277 #modxbughunt
@Mark-H
Copy link
Collaborator

Mark-H commented Jun 15, 2017

Included in 2.6

@Mark-H Mark-H closed this as completed Jun 15, 2017
wuuti pushed a commit to wuuti/revolution that referenced this issue Feb 7, 2018
@joeke
Fixed issue modxcms#13277 #modxbughunt
66252bf
wuuti pushed a commit to wuuti/revolution that referenced this issue Feb 7, 2018
@joeke
Fixed issue modxcms#13277 #modxbughunt
c0c0197
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
proposal Proposal about improvement aka RFC. Need to be discussed before start implementation. type-frontend Issues related to UI/UX issues, mostly about styles and frontend implementations on JavaScript.
Projects
None yet
Milestone
v2.6.0
Development

No branches or pull requests
4 participants
@Jako @Mark-H @jaygilmore @joeke