blinkk / rootjs

Root.js – A full-featured web development tool with a built-in CMS.
https://rootjs.dev
MIT License
7 stars 0 forks source link

feat: add publishing lock #343

Closed stevenle closed 4 months ago

stevenle commented 4 months ago

fixes #341

screens:

Screenshot 2024-06-03 at 11 14 58 PM Screenshot 2024-06-03 at 11 15 18 PM Screenshot 2024-06-03 at 11 15 25 PM Screenshot 2024-06-03 at 11 15 30 PM Screenshot 2024-06-03 at 11 21 32 PM
stevenle commented 4 months ago

Yeah, I've been thinking about how best to incorporate an approval flow into the UI in a way that's intuitive. My thinking right now is that if an approval config is enabled, we would force all publishing to go through the "Release" process and the "Publish" button on the document page would be changed to "Add to release" (or "Go to release"). The release can contain 1 or more doc, a description of the release, and snapshots of before/after for each doc in the release. This would be similar to a GitHub PR - we can maybe even enable comments by allowing people to comment within the snapshot.

Within this release review system, we can enable "presubmit checks" like you're suggesting (e.g. checking for blocked terms).