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’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add a "notes pane" to the edit screen #52

Open
2 tasks
phae opened this issue Aug 10, 2016 · 13 comments
Open
2 tasks

Add a "notes pane" to the edit screen #52

phae opened this issue Aug 10, 2016 · 13 comments

Comments

@phae
Copy link
Contributor

phae commented Aug 10, 2016

User needs:

  • a convenient place to see what's happened with a post over time, in context

As an MVP of basic discussion on PRs, add a panel to the side of the edit screen to house information.

For now, just list the major actions. See: https://phae.github.io/netlify-sketch/review.html (the coloured boxes in particular showing status changes) Commenting etc. can go in a seperate issue.

Acceptance criteria:

  • User can click a link marked "Notes" from a tab menu above the preview pane.
  • In most-recent-first order, a user can see who created the post first, who reviewed it and who published it.
@erquhart
Copy link
Contributor

erquhart commented Jan 3, 2018

We should re-open this discussion based on the current CMS interface.

A few thoughts:

  • Tracking history, notes, and status changes would be awesome!
  • This can become valuable data for any serious organization - how can we make it redundant (able to be regenerated)
  • How can we integrate this into the editor UI without being intrusive?
  • We can probably roll Add notes for drafts #53 into this.

@mittalyashu
Copy link
Contributor

What's the plan here: Will there be new notes for a new change or the note will be same for each item in the collections.

What I think, instead of adding some kind of option for note taking. There should be an option to write custom commit messages.

Reference: Commit Message Template

@stale
Copy link

stale bot commented Oct 29, 2019

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 stale bot added the status: stale label Oct 29, 2019
@rdebeasi
Copy link
Contributor

Not stale. Thank you!!

@stale stale bot removed the status: stale label Oct 31, 2019
@erquhart
Copy link
Contributor

erquhart commented Jan 8, 2020

Clarifying: this is basically a request for pull request comments to be readable/writable through the CMS for unpublished changes, with some additional info that's also available in pull request metadata from any Git host (Eg., who created it and when).

@erquhart erquhart added the pinned label Jan 8, 2020
@sidag95
Copy link

sidag95 commented Apr 4, 2021

Hi @erquhart, I just wanted to know what is the priority of this issue and whether the team is planning to pick this up anytime soon?

@erezrokah
Copy link
Contributor

Hi @sidag95, this is not prioritized yet. If anyone would like to submit a contribution for it that would be great.

@sidag95
Copy link

sidag95 commented Apr 5, 2021

Hi @erezrokah would love to contribute towards this. Will have a look at the Contributions guide. Apart from that, where should I start?

@erezrokah
Copy link
Contributor

Thats great @sidag95. The contribution guide is a great start.

I would start by outlining the proposed UI changes in this issue, after that we would need to implement the comments/notes support in every backend (see https://github.com/netlify/netlify-cms/tree/master/packages/netlify-cms-backend-github for example).

@sidag95
Copy link

sidag95 commented Apr 5, 2021

Got it. Let me look at the contribution guide and propose the UI changes first and take it from there. Thanks!

@sidag95
Copy link

sidag95 commented Apr 24, 2021

Looking at all the features that can be supported by having a notes panel, I am proposing that we should start with a basic MVP that allows users to leave comments on each post. The MVP can looking something like below.

notes-phase1

Once we have implemented this with all the backends, we should be able to extend this to support history, add reviewers and more.

Open questions

  1. Should the notes panel respect i18n?

@erezrokah Does this make sense? How should we proceed from here?

@erezrokah
Copy link
Contributor

Hi @sidag95, this is great progress 🎉

  1. Should the notes panel respect i18n?

Yes, I think adding a i18n tab would be good.

@erezrokah Does this make sense? How should we proceed from here?

I would like to see an approach to implement this. I'm assuming PR comments, but making those look "realtime" can be a challenge. I know GitHub uses WebSockets for that, but it's not an official API. We could use polling/and or add a refresh button, we'll just to need to aware of performance and possibly services rate limits.

@sidag95
Copy link

sidag95 commented Apr 26, 2021

Yes, I think adding a i18n tab would be good.

Okay, will check how the current i18n window looks like and make changes appropriately

performance and possibly services rate limits.

I believe this notes window doesn't need to be realtime. We can start with a large polling window and give a refresh button to the users so they have explicit control over the recency of notes/comments.

Let me setup some code and start a small PR. We can continue feature specific discussions there.

Thanks for the support @erezrokah!

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

No branches or pull requests

7 participants