Follow-up reviews in MozReview are hard (doesn't show which diffs I've already reviewed). Can lead to missed code.

RESOLVED DUPLICATE of bug 1244884

Status

MozReview
General
RESOLVED DUPLICATE of bug 1244884
3 years ago
2 years ago

People

(Reporter: jib, Unassigned)

Tracking

(Depends on: 1 bug)

Details

STR:
1. Go to Bug 1208371 comment 361 from me. Here I have 12 review requests.
2. Try to determine which interdiff #s I need to review for each of the 12 patches
   (My last review here is from around Bug 1208371 comment 205).

I've spent over half an hour now determining which diff versions (I think it's 2-5? though maybe they vary by patch?) that I need to be reviewing. It might help if MozReview kept track of the last diffs I've reviewed, or maybe showed associated dates for the various numeric (1,2,3,4,5 etc.) "revisions"...?

I gather the interdiff links that are auto-posted to bugzilla are meant to help, but here they're actually trouble, because had I followed them I would have r+'ed based solely on the last interdiff, missing lots of code!

The other problem with the interdiff links is that they're in bugzilla rather than in mozReview itself.

It seems to me that mozReview has all the information it needs to present exactly what I need to review.
(Assignee)

Updated

2 years ago
Product: Developer Services → MozReview
I had the same problem today and found bug 1296672 which might help here.
Depends on: 1296672
Duping to older since it was fixed there.
Status: NEW → RESOLVED
Last Resolved: 2 years ago
Resolution: --- → DUPLICATE
Duplicate of bug: 1244884
You need to log in before you can comment on or make changes to this bug.