Feature #1725
closedDelete button on comments
Added by Derek Montgomery about 16 years ago. Updated over 8 years ago.
0%
Description
I know people with rights can just remove all content of a comment to get rid of it but it is not as "evident" as a delete icon, especially when quote and edit icons are present.
Files
add_a_delete_link_to_journal_entries.patch (968 Bytes) add_a_delete_link_to_journal_entries.patch | Adds a delete button to issue journal entries | Felix Schäfer, 2010-07-07 20:00 | |
delete-note.png (47.3 KB) delete-note.png | screenshot | Go MAEDA, 2016-01-17 16:24 | |
add_a_delete_link_to_journal_entries_v2.patch (1.67 KB) add_a_delete_link_to_journal_entries_v2.patch | patch against 3.2.0.devel.15067 | Go MAEDA, 2016-01-17 16:25 |
Related issues
Updated by Felix Schäfer about 14 years ago
- Assignee set to Felix Schäfer
I've been wondering about this too, will have a look at it and see how quickly feasible it is.
Updated by Felix Schäfer about 14 years ago
- File add_a_delete_link_to_journal_entries.patch add_a_delete_link_to_journal_entries.patch added
- Status changed from New to Resolved
- Target version set to 1.0.1
The supplied patch adds a delete button to an issue's journal entries, only for users with the right to edit the journal entry.
Updated by Eric Davis almost 14 years ago
- Assignee changed from Felix Schäfer to Eric Davis
- Target version deleted (
1.0.1)
There is a problem with this patch. Journals are only deleted if:
- The notes are empty and
- The details are empty
All this patch does is to clear the notes. If a Journal had details, it wouldn't be deleted. There are two options we can do here:
- Add a new action to really delete a Journal and provide a link to it, or
- Change this icon so it's clear that it only clears the note.
I've added Holger to this issue, I know he has an opinion about deleting things like Journals and I would like to hear what he thinks.
Updated by Felix Schäfer almost 14 years ago
The scope of the patch wasn't to delete journals but the notes attached to it, which is currently possible to if you edit the notes to an empty string. The current functions quote and edit only work on the notes too, so I don't think that would be that misleading.
Updated by Eric Davis almost 14 years ago
Maybe the title attribute should say "Delete notes" then. What do you think?
Updated by Felix Schäfer almost 14 years ago
- Assignee changed from Eric Davis to Felix Schäfer
Can do that, I'll change the other titles (edit and quote) too to reflect that if it's ok with you?
Updated by Felix Schäfer almost 14 years ago
- Assignee changed from Felix Schäfer to Eric Davis
Eric, I slightly reworked the patch to have the title say "Delete notes", the confirmation pop-up now also states that the changes to the issue will not be reverted if the journal details are not empty.
See http://github.com/thegcat/redmine/commit/5be3c6e9b405bf03ad69226e8235e3d6618adbf5 for the change.
Updated by Brian Lindahl over 13 years ago
created a related patch for rolling back journal entries #7610
This rolls back the changes made to the issue as well as removing the journal entry. Note that this capability is only enabled for the last journal entry (for obvious reasons).
Updated by Jean-Philippe Lang over 11 years ago
- Status changed from 7 to New
Assigned issue with no assignee back to New status.
Updated by rm user over 10 years ago
what's up with this? (5 years old and counting...)
i managed to delete WHOLE issue instead of the single comment.. :(
Updated by Go MAEDA over 8 years ago
- File delete-note.png delete-note.png added
- File add_a_delete_link_to_journal_entries_v2.patch add_a_delete_link_to_journal_entries_v2.patch added
- Target version set to 3.3.0
Updated patch for 3.2.0.devel.15067.
The procedure to delete note is a little bit difficult to understand. This change improve usability. In addition, we can avoid accidental deletion of a issue by clicking delete button under a latest note.
Please include this feature in Redmine 3.3.0.
Updated by Jean-Philippe Lang over 8 years ago
- Status changed from New to Closed
- Assignee set to Jean-Philippe Lang
- Resolution set to Fixed
Updated by rm user over 8 years ago
wow! 7years old issue is finally solved!
big up Jean-Philippe and Go MAEDA!
what about permissions btw? is there a permission where user can delete only his own comments on issues?
Updated by Jean-Philippe Lang over 8 years ago
rm user wrote:
what about permissions btw? is there a permission where user can delete only his own comments on issues?
As noted above, users with "edit notes" or "edit own notes" were already able to delete notes by making them empty. The delete button is just some kind of shortcut for doing this. The permissions are the same, so the answer is yes, you can allow users to edit/delete only their own notes.