Hi, This is currently not supported. We've done a fair amount of investigation into what it would take to display old comments on new versions of the diff, and it's a very difficult problem to do well. We'll continue thinking about this, but for the time being, you'll just have to go "back and forth"
-David On Mon Feb 02 2015 at 8:58:02 AM Nitin Agrawal <[email protected]> wrote: > Hi, > > Here I am facing a problem as a reviewer. Below are the steps (Problem is > in Step 8): > > 1) Suppose one review request comes to me (Reviewer 1) for review. > 2) I have reviewed the code and gave some comments on suppose line numbers > X, Y, Z. > 3) Second reviewer (Reviewer 2) also gave the comments on same file but > different line numbers A, B, C. (In our organization, one review can go to > multiple reviewers.) > 4) Now the developer of the code, incorporated the changes related to > review comments of both Reviewer 1 and Reviewer 2. > 5) After that the developer updates the diff on same review ID and publish > it. > 6) After publishing, a mail comes to me (Reviewer 1) and I open the link > of review board provided in that mail. > 7) When the page opens, I click on "View Diff" link on the top right > corner to see the diff. > 8) The problem here is, I am not able to see the comments given by me (in > step 2) on "View Diff" page. Although I can see the "diff" but not able to > see the review comments therefore I am not able to find out whether the > developer has made the changes what I wanted or something different (as I > am not able to see my comments). > 9) For proper reviewing the changes, what I am doing right now is doing > "Back and Forth". Means I am reading line numbers and my comments from my > previous review mail and verifying in the current diff which is very time > consuming and boring. > > Here what I want to understand that are below changes possible inside > review board to make a reviewer's life easier ? > > 1) As I mentioned in step 8, is it possible to show the my (Reviewer 1's) > review comments in the current diff? > 2) If above is possible, please also show review comments by Reviewer 2 > but with different color so that Reviewer 1 and Reviewer 2 can check only > their corresponding comments. > > Thanks in Advance > > -- > Supercharge your Review Board with Power Pack: > https://www.reviewboard.org/powerpack/ > Want us to host Review Board for you? Check out RBCommons: > https://rbcommons.com/ > Happy user? Let us know! https://www.reviewboard.org/users/ > --- > You received this message because you are subscribed to the Google Groups > "reviewboard" group. > To unsubscribe from this group and stop receiving emails from it, send an > email to [email protected]. > For more options, visit https://groups.google.com/d/optout. > -- Supercharge your Review Board with Power Pack: https://www.reviewboard.org/powerpack/ Want us to host Review Board for you? Check out RBCommons: https://rbcommons.com/ Happy user? Let us know! https://www.reviewboard.org/users/ --- You received this message because you are subscribed to the Google Groups "reviewboard" group. To unsubscribe from this group and stop receiving emails from it, send an email to [email protected]. For more options, visit https://groups.google.com/d/optout.
