Add file attachment comments to diff comments hint.

Review Request #13684 — Created April 1, 2024 and submitted

Information

Review Board
release-7.x

Reviewers

Now that we have file attachment comments showing up as part of the
diff, we also want to include that data in the diff comments hint. This
change adds that in, expanding the file attachment comment helper
function to return some additional data useful for this.

This simplifies the client side a bit by allowing the tip revision to be
null, for the case where we don't want the URL to include
?tip-commit-id.

Created a draft review with a bunch of comments on files in various diff
revisions, interdiff revisions, and commit ranges (as well as some
comments on regular file attachments). Saw that the diff comment hint
correctly contained all the revisions and that clicking on everything
loaded the correct page.

Summary ID
Add file attachment comments to diff comments hint.
Now that we have file attachment comments showing up as part of the diff, we also want to include that data in the diff comments hint. This change adds that in, expanding the file attachment comment helper function to return some additional data useful for this. This simplifies the client side a bit by allowing the tip revision to be null, for the case where we don't want the URL to include `?tip-commit-id`. Testing Done: Created a draft review with a bunch of comments on files in various diff revisions, interdiff revisions, and commit ranges (as well as some comments on regular file attachments). Saw that the diff comment hint correctly contained all the revisions and that clicking on everything loaded the correct page.
9aab363bbd3f3b593bd6e674dfd5e8cda720526e
Description From Last Updated

Can we set this alphabetically?

chipx86chipx86
chipx86
  1. 
      
  2. reviewboard/reviews/views/diffviewer.py (Diff revision 1)
     
     
     
     
     
     
    Show all issues

    Can we set this alphabetically?

  3. 
      
david
chipx86
  1. Ship It!
  2. 
      
maubin
  1. Ship It!
  2. 
      
david
Review request changed

Status: Closed (submitted)

Change Summary:

Pushed to release-7.x (269c2ec)
Loading...