HomePhorge

Display Reviewed By instead of first Reviewer in revisions overview

Description

Display Reviewed By instead of first Reviewer in revisions overview

Summary:
Displaying reviewer who was by coincidence listed first is quite confusing, especially for committed revisions.
This displays the one who really reviewed the revision if available.

This implementation is pretty bad from performance perspective - O(N) queries to retrieve all comments.
The page load still feels quite fast.

Test Plan: /differential/filter/revisions/

Reviewers: epriestley

CC: aran, epriestley

Differential Revision: https://secure.phabricator.com/D1772

Details

Provenance
vranaAuthored on Mar 4 2012, 2:44 AM
themackabuPushed on Mar 25 2025, 8:07 PM
Parents
rPf2caa6888eea: Simplify Owners interfaces to Audit
Branches
Unknown
Tags
Unknown

Event Timeline