You cannot select more than 25 topics
Topics must start with a letter or number, can include dashes ('-') and can be up to 35 characters long.
Zettat123
7f790c70b9
Remove delete button for review comment (#23036)
Fix #23031.
Currently, only comments with type `CommentTypeComment` or
`CommentTypeCode` can be deleted. If user create a review comment, the
type of the comment will be `CommentTypeReview` so the comment cannot be
deleted.
e7be610d57/routers/web/repo/issue.go (L2860-L2868)
And in Github, user also cannot delete a review comment. There isn't a
delete button in the menu.
<img
src="https://user-images.githubusercontent.com/15528715/220275166-5ae2dc10-4003-4857-b14e-d7b02644345f.png"
width="640px" />
So we should remove the delete button from the menu when the comment's
type is `CommentTypeReview`.
|
2 years ago |
| .. |
|
admin
|
Refactor hiding-methods, remove jQuery show/hide, remove `.hide` class, remove inline style=display:none (#22950)
|
2 years ago |
|
api/packages/pypi
|
Add Package Registry (#16510)
|
3 years ago |
|
base
|
Introduce customized HTML elements, fix incorrect AppUrl usages in templates (#22861)
|
2 years ago |
|
code
|
Remove Fomantic-UI's `.hidden` CSS class for menu elements (#22895)
|
2 years ago |
|
custom
|
…
|
|
|
explore
|
Add context cache as a request level cache (#22294)
|
2 years ago |
|
mail
|
Use link in UI which returned a relative url but not html_url which contains an absolute url (#21986)
|
2 years ago |
|
org
|
Hide 2FA status from other members in organization members list (#22999)
|
2 years ago |
|
package
|
Introduce customized HTML elements, fix incorrect AppUrl usages in templates (#22861)
|
2 years ago |
|
projects
|
Scoped labels (#22585)
|
2 years ago |
|
repo
|
Remove delete button for review comment (#23036)
|
2 years ago |
|
shared
|
Fix no user listed in org teams page (#22979)
|
2 years ago |
|
status
|
Move helpers to be prefixed with `gt-` (#22879)
|
2 years ago |
|
swagger
|
Add scopes to API to create token and display them (#22989)
|
2 years ago |
|
user
|
Add scopes to API to create token and display them (#22989)
|
2 years ago |
|
home.tmpl
|
Add main landmark to templates and adjust titles (#22670)
|
2 years ago |
|
install.tmpl
|
Refactor hiding-methods, remove jQuery show/hide, remove `.hide` class, remove inline style=display:none (#22950)
|
2 years ago |
|
post-install.tmpl
|
Introduce customized HTML elements, fix incorrect AppUrl usages in templates (#22861)
|
2 years ago |
