Skip to content

Conversation

@6543
Copy link
Member

@6543 6543 commented Oct 6, 2019

Backport of #7917

@6543
Copy link
Member Author

6543 commented Oct 6, 2019

@lunny @techknowlogick backport ...
... drone done

@GiteaBot GiteaBot added the lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. label Oct 6, 2019
@6543
Copy link
Member Author

6543 commented Oct 7, 2019

@guillep2k point

@GiteaBot GiteaBot added lgtm/need 1 This PR needs approval from one additional maintainer to be merged. and removed lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. labels Oct 7, 2019
@lunny lunny added the type/bug label Oct 7, 2019
@lunny lunny added this to the 1.9.4 milestone Oct 7, 2019
@GiteaBot GiteaBot added lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore. and removed lgtm/need 1 This PR needs approval from one additional maintainer to be merged. labels Oct 7, 2019
@techknowlogick
Copy link
Member

ping LG-TM bot

@techknowlogick techknowlogick merged commit 30718ce into go-gitea:release/v1.9 Oct 7, 2019
@6543 6543 deleted the backports_7917 branch October 7, 2019 09:28
@go-gitea go-gitea locked and limited conversation to collaborators Nov 24, 2020
project-mirrors-bot-tu bot pushed a commit to project-mirrors/forgejo-as-gitea-fork that referenced this pull request Oct 11, 2025
Resolves go-gitea#8403

There's a issue with the comment history items that are deleted, they can be deleted by the author of the comment or by those that have write access. However when the item is deleted, it shows that the one who wrote that revision of the comment deleted it, which is not always true. Strike-through the item instead.

Co-authored-by: Daniele Pintore <[email protected]>
Co-authored-by: 0ko <[email protected]>
Co-authored-by: Gusted <[email protected]>
Reviewed-on: https://codeberg.org/forgejo/forgejo/pulls/8458
Reviewed-by: 0ko <[email protected]>
Co-authored-by: danielep <[email protected]>
Co-committed-by: danielep <[email protected]>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore. type/bug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants