tuxmain/webcomment
1
0
Fork 0
Code Issues 20 Pull requests Projects Releases Packages Wiki Activity

Comment edit #6

New issue
Open
opened 2022-10-15 13:35:41 +00:00 by tuxmain · 0 comments
tuxmain commented 2022-10-15 13:35:41 +00:00
Owner
Copy link
  • admin edit
  • author edit

A comment can be edited only within comment_edit_timeout.

Pending comments can be edited freely.

Approved comments can be edited, but the edited version will be pending, and will replace the old version only when approved.

User interface should allow to remove the old comment when editing a comment, if approval is enabled and the comment is already approved.

TODO: identify comment versions to prevent a pending comment being modified between admin review and admin approval.

To allow future edition, when posting a new comment the server gives a mutation token to the client (the token (or its hash?) is stored with the comment).

* [x] admin edit * [ ] author edit A comment can be edited only within `comment_edit_timeout`. Pending comments can be edited freely. Approved comments can be edited, but the edited version will be pending, and will replace the old version only when approved. User interface should allow to remove the old comment when editing a comment, if approval is enabled and the comment is already approved. TODO: identify comment versions to prevent a pending comment being modified between admin review and admin approval. To allow future edition, when posting a new comment the server gives a mutation token to the client (the token (or its hash?) is stored with the comment).
tuxmain added this to the prod-ready milestone 2022-10-19 17:19:53 +00:00
Sign in to join this conversation.
No Branch/Tag specified
Branches Tags
main
webgui
No results found.
Labels
Clear labels
No items
No labels
Milestone
Clear milestone
No items
No milestone
prod-ready
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: tuxmain/webcomment#6
No description provided.
Delete branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?