1
0
Fork 0
mirror of https://we.phorge.it/source/phorge.git synced 2024-09-21 01:38:48 +02:00
phorge-phorge/src/applications
vrana 87c60abbd0 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
2012-03-05 10:30:36 -08:00
..
audit Make Diffusion Herald emails thread with Audit emails 2012-03-05 09:54:45 -08:00
auth OAuthServer polish and random sauce 2012-03-01 14:46:18 -08:00
base Send 403 for admin pages without being admin 2012-01-15 17:30:23 -08:00
calendar Build a basic calendar view 2011-08-08 10:34:06 -07:00
chatlog Add a basic chatlog 2012-02-17 10:21:38 -08:00
conduit Don't corrupt commit message in arc amend 2012-03-01 22:09:49 -08:00
countdown Use generic URIs 2012-01-16 14:53:05 -08:00
daemon Move functionality of PhabricatorMetaMTADaemon to a worker task 2012-03-01 22:01:55 -08:00
differential Display Reviewed By instead of first Reviewer in revisions overview 2012-03-05 10:30:36 -08:00
diffusion Use wide links only for line and blame-prev in Diffusion 2012-02-29 18:22:04 -08:00
directory Show commits in /audit/ 2012-02-28 21:10:39 -08:00
draft/storage Revision comment drafts. 2011-02-05 16:57:21 -08:00
drydock Use setConcreteOnly() in Phabricator and only list/launch concrete Daemons 2012-01-25 11:50:59 -08:00
feed Fix Feed Times on 32 Bit Servers 2012-02-28 21:04:02 -08:00
files Display //no name// for files without name to make the link clickable 2012-02-23 13:23:04 -08:00
help/controller Explicitly show that "escape" closes dialogs in Phabricator 2011-08-02 09:21:28 -07:00
herald Allow Herald to trigger audits for users or projects 2012-02-27 09:36:30 -08:00
maniphest Use PHPExcel, not Spreadsheet_Excel_Writer, to export Excel sheets 2012-03-01 17:23:29 -08:00
markup Show audit comments on the Diffusion commit view 2012-02-24 14:14:39 -08:00
metamta Move functionality of PhabricatorMetaMTADaemon to a worker task 2012-03-01 22:01:55 -08:00
oauthserver OAuth Server -- add a doc and link to it in a few places 2012-03-01 17:27:52 -08:00
owners Simplify Owners interfaces to Audit 2012-03-05 09:57:46 -08:00
paste Add line link to Paste 2012-02-08 10:54:57 -08:00
people Remove commit list from Diffusion in favor of Audit commit list 2012-02-28 21:12:08 -08:00
phid Show commits in /audit/ 2012-02-28 21:10:39 -08:00
phriction Add a contextual "scope" dropdown for searches 2012-02-14 17:00:12 -08:00
project Added Additional Fuctionality to Jump Nav: Jump to users, projects, symbols, or 2012-02-20 10:23:51 -08:00
repository Make Diffusion Herald emails thread with Audit emails 2012-03-05 09:54:45 -08:00
search Add audits to search 2012-02-27 09:51:00 -08:00
slowvote Kill PhabricatorFileURI 2012-01-10 15:21:39 -08:00
status/base Add /status/ 2011-04-08 11:13:51 -07:00
typeahead/controller Enhance Maniphest custom queries 2012-02-28 21:08:02 -08:00
uiexample Examples using JX.View 2011-11-06 15:17:00 -08:00
xhpastview Add missing includes from XHPAST parse bug. 2011-04-06 23:14:58 -07:00
xhprof Fix iframe issue for XHProf DarkConsole plugin 2012-01-28 11:17:19 -08:00