Ability to delete a comment

Labels: area/comments kind/feature lifecycle/stale priority/backlog triage/accepted

Timeline

Martin Lindhe (martinlindhe) opened

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 ".".

Martin Lindhe (martinlindhe) changed the title from Ability to delete a comment to Ability to delete a comment

Michael Muré (MichaelMure) commented

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.

Martin Lindhe (martinlindhe) commented (edited)

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

Michael Muré (MichaelMure) added label good first issue

Michael Muré (MichaelMure) added label Core

Michael Muré (MichaelMure) added label Easy pick

github-actions (github-actions) commented

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 (github-actions) added label lifecycle/stale

sudoforge removed label good first issue

sudoforge removed label Core

sudoforge removed label Easy pick

sudoforge removed label lifecycle/stale

sudoforge added label kind/feature

sudoforge added label area/comments

sudoforge added label priority/backlog

sudoforge added label triage/accepted

github-actions (github-actions) commented

This bot triages issues in order to help the maintainers identify what needs attention, according to the following lifecycle rules:

  • After 90 days of inactivity, lifecycle/stale is applied
  • After 90 days of inactivity since lifecycle/stale was applied, lifecycle/rotten is applied

This bot will not automatically close stale issues.

To remove the stale status, you can:

  • Remove the stale label from this issue
  • Comment on this issue
  • Close this issue
  • Offer to help out with triaging

To avoid automatic lifecycle management of this issue, add lifecycle/frozen.

github-actions (github-actions) added label lifecycle/stale

Michael Muré (MichaelMure) commented

Not stale

On Fri, Nov 1, 2024, 4:29 AM github-actions[bot] @.***> wrote:

This bot triages issues in order to help the maintainers identify what needs attention, according to the following lifecycle rules:

  • After 90 days of inactivity, lifecycle/stale is applied
  • After 90 days of inactivity since lifecycle/stale was applied, lifecycle/rotten is applied

This bot will not automatically close stale issues.

To remove the stale status, you can:

  • Remove the stale label from this issue
  • Comment on this issue
  • Close this issue
  • Offer to help out with triaging

To avoid automatic lifecycle management of this issue, add lifecycle/frozen.

— Reply to this email directly, view it on GitHub https://github.com/git-bug/git-bug/issues/939#issuecomment-2451225548, or unsubscribe https://github.com/notifications/unsubscribe-auth/AACH6DNVAO6PTHM3VV6R3DLZ6LYR7AVCNFSM6AAAAABLJPS4NSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDINJRGIZDKNJUHA . You are receiving this because you commented.Message ID: @.***>

sudoforge removed label lifecycle/idle

sudoforge added label lifecycle/pinned

sudoforge removed label lifecycle/pinned