git-bug icon indicating copy to clipboard operation
git-bug copied to clipboard

Ability to delete a comment

Open martinlindhe opened this issue 2 years ago • 3 comments

I would like to "delete" a comment.

In the sense of marking it as deleted, so that in it's place, i would see "Comment deleted by Name" rather than the content.

If i try to edit the comment and remove it's text, i get "Empty message, aborting" in termui.

If i do the same in the webui, the "update comment" button gets grayed out.

Currently I work around this by editing comment and replacing content with ".".

martinlindhe avatar Dec 02 '22 18:12 martinlindhe

That would be a great addition. That said, by the nature of git-bug keeping a full history, a deleted comment could still be "retrieved from the past", a large difference with centralized bugtrackers. In that sense, we might as well keep having the full history in the UIs.

Would you like giving it a try? It should be easy enough as that would be adding another operation type very similar to EditCommentOperation.

MichaelMure avatar Dec 02 '22 19:12 MichaelMure

That would be a great addition. That said, by the nature of git-bug keeping a full history, a deleted comment could still be "retrieved from the past", a large difference with centralized bugtrackers. In that sense, we might as well keep having the full history in the UIs.

Yea, not really deleting just marking comments as "deleted".

I imagine in the web ui for example, you could just list the deletion as "comment marked as deleted" in the change history, where the older entry is the full text of the comment.

Would you like giving it a try? It should be easy enough as that would be adding another operation type very similar to EditCommentOperation.

I'm tempted but just started evaluating your software in this other project I'm knee deep in at the moment. I'll keep subscribed and maybe I'll get back to it

martinlindhe avatar Dec 02 '22 21:12 martinlindhe

This bot triages untriaged issues and PRs according to the following rules:

  • After 90 days of inactivity, the lifecycle/stale label is applied
  • After 30 days of inactivity since lifecycle/stale was applied, the issue is closed

To remove the stale status, you can:

  • Remove the lifecycle/stale label
  • Comment on this issue

github-actions[bot] avatar Jul 23 '24 03:07 github-actions[bot]