decap-cms icon indicating copy to clipboard operation
decap-cms copied to clipboard

Bundle editorial workflow changes into one review branch

Open ebello opened this issue 7 years ago • 13 comments

- Do you want to request a feature or report a bug? Feature request

- What is the current behavior? Currently every page updated or created within an editorial workflow setup gets applied as a separate pull request. The current workflow when hosting with Netlify is:

  1. Single page edited or created within CMS.
  2. Pull request created on Github to manage changes for that page.
  3. Deploy automatically started on Netlify for that one change.
  4. Deploy notification sent with URL showing just that one change.

- What is the expected behavior? It would be great to have a way within the CMS to combine changes so that they can be reviewed together in context before going live. A new workflow could be:

  1. On editorial workflow page, provide button that says "Review changes".
  2. When button is clicked, the option to select individual changes to review via the cards can be presented.
  3. A prompt is provided to label these changes.
  4. When confirmed, a new branch is created using the label name, and all the pull requests selected via the cards will be merged into the new branch.
  5. Deploy automatically started on Netlify for that new branch.
  6. Deploy notification sent with URL showing all changes selected.

It would be good to maintain a list of branches created this way in the editorial page UI so that any further changes can be remerged into the same branch, or have the branch eventually merged with master and then delete the branch as well as close all related pull requests.

ebello avatar Jan 18 '18 18:01 ebello

I'd love to see this, we've actually discussed it a lot in the past. It's common to have multiple pieces of content that all need to be reviewed and published as a group, agreed.

Maintaining a list of available content groups (branches underneath) and allowing them to be assembled in the UI makes a ton of sense - I think the biggest challenge will be how we express it in the UI. The current column approach may not be best, definitely worth discussing.

cc/ @neutyp @ziburski

erquhart avatar Jan 20 '18 15:01 erquhart

I had a thought that the current three column approach could be refined like so:

In Review

  • Rather than one long column it could consist of groupings of drafts
  • Include an ever present button under the header that says “New review group” (not sold on this labeling but you get the idea)
  • Individual changes can be dragged into a review group from the drafts column or another review group
  • When a review group is ready for publishing you can drag the entire group to the Ready column
  • If a grouping is deleted all changes get moved back to the drafts column

Ready

  • Instead of publishing individual changes there could be a button under the header that says “Publish all changes” that would automatically merge all changes in the Ready column into master

I realize this would have side effects like not being able to update the status for an individual change, but I think it’s worth the sacrifice for this greater flexibility.

ebello avatar Jan 22 '18 04:01 ebello

react-beautiful-dnd is a higher level library than our current one (react-dnd), and I've been wanting to migrate to it for some time. It includes drag and combine functionality - dropping an item onto another item to create a group - that would fit the bill perfectly: https://react-beautiful-dnd.netlify.com/?selectedKind=multiple%20vertical%20lists&selectedStory=stress%20test&full=0&addons=0&stories=1&panelRight=0

A few other issues that are closely connected to this:

  • https://github.com/netlify/netlify-cms/issues/1344
  • #192

erquhart avatar Feb 11 '19 22:02 erquhart

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

stale[bot] avatar Oct 29 '19 11:10 stale[bot]

Hello, I am "upvoting" for the feature "Publish all changes" on editorial workflow.

I've open a topic on the Netlify Community with this question/solution for the button. You can take a look at: https://community.netlify.com/t/publish-all-changes-button-on-editorial-workflow-avoid-multiple-deploys/9958

ositaka avatar Mar 04 '20 09:03 ositaka

Wauw almost three years ago this feature request was openend and still nothing really? My client is so happy :(

timothy-kodes avatar Dec 04 '20 14:12 timothy-kodes

If this is too hard to implement, maybe offer us a config option (another value for publish_mode maybe?) that uses editorial_workflow, but re-uses the same branch, draft, for all draft edits. And it would also have to use branches for in review and ready, instead of labels.

schindld avatar Dec 11 '20 17:12 schindld

Any update? I just started using this, but am wondering if I can just disable automatic deployments and deploy each change one by one to not avoid repetitive failed builds?

hstandeffer avatar Feb 14 '21 07:02 hstandeffer

What's the state of this issue? It's definitely a very desired feature in our team.

alexgonch avatar Sep 21 '21 23:09 alexgonch

Any news yet? I really think this is a must.

inavarrorubio avatar Aug 30 '22 08:08 inavarrorubio

This is critical. If anyone is planning to make progress on this, let's collaborate.

mohab-sameh avatar Oct 05 '22 11:10 mohab-sameh

I feel like this would improve the cms experience greatly. 👍 for optimizing the process to save time and money

marijanasevo avatar Aug 02 '23 20:08 marijanasevo

Upvote for this feature. Would save on build time so much.

brandnewguys avatar Oct 05 '23 12:10 brandnewguys