flake8
-
reviewboard/reviews/ui/base.py (Diff revision 1) Show all issues
Review Request #13329 — Created Oct. 11, 2023 and submitted
Our current UI allows users to comment on deleted file attachments, but these
comments cannot actually be saved for the deleted files. It also doesn't make
sense to allow commenting on deleted files. To address this, this change blocks
commenting on deleted files. When attempting to comment on one, we display a
warning message in the comment dialog and block the ability to edit the file,
similar to what we display when unauthenticated users try to create comments.
We update theFileAttachmentCommentBlock
andFileAttachmentReviewable
models to make use of the file attachment states concept that were recently
introduced.This also hides the Markdown docs link for comment dialogs that cannot be
edited. This makes more sense than hiding it only for unauthenticated
users, which is what we were doing before.
Summary | ID |
---|---|
b81b87818168fbbcc542351d6a90fa8590bdf8b1 |
Removed unused import.
Commits: |
|
|||||||
---|---|---|---|---|---|---|---|---|
Diff: |
Revision 2 (+320 -112) |
Code looks good! Just want to get some version info in the docs.
reviewboard/static/rb/js/models/abstractCommentBlockModel.es6.js (Diff revision 2) |
---|
Can you add a
Version Added
?
reviewboard/static/rb/js/models/fileAttachmentCommentBlockModel.es6.js (Diff revision 2) |
---|
Can you add a
Version Added
?
reviewboard/static/rb/js/reviews/views/commentDialogView.ts (Diff revision 2) |
---|
Can you add a
Version Added
?
reviewboard/static/rb/js/reviews/views/commentDialogView.ts (Diff revision 2) |
---|
Can you add a
Version Added
?
Commits: |
|
|||||||
---|---|---|---|---|---|---|---|---|
Diff: |
Revision 3 (+344 -112) |