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

Allow multiple snapshots to be merged into a single snapshot #67

Closed
westonruter opened this Issue Jul 27, 2016 · 6 comments

Comments

Projects
None yet
3 participants
@westonruter
Contributor

westonruter commented Jul 27, 2016

Inverse of #16.

Probably should create a new snapshot post post that contains the merged settings from the other snapshots. We could use bulk actions to implement this.

@westonruter

This comment has been minimized.

Contributor

westonruter commented Jul 27, 2016

We could potentially trash the snapshots that are merged into the single snapshot. This would also entail preventing snapshots that are published or pending from being available for merging.

@kienstra

This comment has been minimized.

Contributor

kienstra commented Aug 17, 2016

Hi @westonruter,
Would there be a way to handle conflicting settings in merged snapshots?

@westonruter

This comment has been minimized.

Contributor

westonruter commented Aug 17, 2016

@kienstra great question. I suppose so! I guess there would need to be an intermediate step where any conflicts are resolved.

@PatelUtkarsh

This comment has been minimized.

Collaborator

PatelUtkarsh commented Oct 6, 2016

@westonruter Should we allow schedule snapshot to be merged?

@PatelUtkarsh

This comment has been minimized.

Collaborator

PatelUtkarsh commented Oct 6, 2016

Also, If a user selects 2 publish and one draft for merge we will show error, but in the case of 1 publish and 2 drafts, Should we process with the merge of 2 draft and show error or just display error without merging it?

@westonruter

This comment has been minimized.

Contributor

westonruter commented Oct 7, 2016

@PatelUtkarsh Yes. I see no reason why a scheduled or even a published snapshot should not be able to be merged. You should be able to include published snapshots in the merges in the same way that you should be able to duplicated/forked (#41). This is closely related to allowing a snapshot to be reverted (#20): consider publishing a snapshot, making some changes, and then wanting to restore the site to a previous state. This could be done by duplicating a previously-published snapshot, or selecting multiple snapshots to merge together to create such a “revert” snapshot.

@PatelUtkarsh PatelUtkarsh referenced this issue Oct 26, 2016

Merged

Snapshot merge resolve conflicts #93

2 of 2 tasks complete

@westonruter westonruter added this to the 0.6.0 milestone Jul 5, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment