Delete a message in the conversation

Also, I noticed that anyone in a thread has permission to delete the thread. That seems pretty aggressive, no?

Deleting a thread only hides messages for the thread member who "deleted" it.
The actual thread entity is deleted when all the members "deleted" it.
There were changes in #3179135: Delete Conversations as Admin → related to that

Looks good for me (added 2 minor comments to the MR), however the new feature should go to the 3.x branch

Thanks for the review @artem_sylchuk.

Please re-review!

Hey, there is a small note on the MR.
I'll merge it once it is resolved, however I see some issues related to this functionality.
As an admin I see the contextual delete link, however I don't see it as a regular user even I still can access the delete url directly.

Also there is some inconsistency, the Thread entity doesn't use ContentEntityDeleteForm as the base class for its deletion form and it cause some difference in log messages and how the deletion forms look (but probably that's fine as threads deletion work a bit differently).

Another small issue is the lack of private message label() method. The message deletion confirmation message uses message id now and it looks not really good.

However all the above except the note in MR should be addressed as the separate issues in my opinion.

Awesome, merged. Great work!

Automatically closed - issue fixed for 2 weeks with no activity.

Hi,
I can see the delete message link only as admin?
Is this the intention to show private message delete button only in the contextual links?