Page MenuHomePhabricator

Make "Active" the default query for People
AbandonedPublic

Authored by cburroughs on Sep 25 2014, 9:16 PM.
Tags
None
Referenced Files
F13226615: D10565.diff
Sun, May 19, 9:58 PM
F13199813: D10565.id25379.diff
Mon, May 13, 6:49 PM
F13199675: D10565.id.diff
Mon, May 13, 5:09 PM
F13195964: D10565.id25379.diff
Sun, May 12, 10:47 PM
F13191196: D10565.id30794.diff
Sat, May 11, 5:57 PM
F13190031: D10565.diff
Sat, May 11, 7:11 AM
F13189538: D10565.id30794.diff
Sat, May 11, 6:09 AM
Unknown Object (File)
May 4 2024, 3:54 AM

Details

Reviewers
epriestley
Group Reviewers
Blessed Reviewers
Summary

On corporate installs the number of former employees can easily rival current ones. The results in the "All" view for the People app showing a lot of noise. Create a new "Active" query showing non-disabled users, and make that the default.

Test Plan

On an install with disabled users:

  • Go to /people/ and see the nice highlight for "Active".
  • Look at the results and verify that none of them are disabled.
  • Click on "All" and verify disabled users are still there.
  • Explicitly click on "Active" and see that it's no different than default.

Diff Detail

Repository
rP Phabricator
Branch
active-people
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 5926
Build 5946: [Placeholder Plan] Wait for 30 Seconds

Event Timeline

cburroughs retitled this revision from to make 'Active' the default query for People.
cburroughs updated this object.
cburroughs edited the test plan for this revision. (Show Details)
joshuaspence edited the test plan for this revision. (Show Details)
joshuaspence edited edge metadata.
joshuaspence retitled this revision from make 'Active' the default query for People to make "Active" the default query for People.Feb 25 2015, 9:43 AM
joshuaspence updated this object.
joshuaspence edited the test plan for this revision. (Show Details)
joshuaspence added a subscriber: joshuaspence.
joshuaspence retitled this revision from make "Active" the default query for People to Make "Active" the default query for People.Feb 25 2015, 9:45 AM
epriestley added a reviewer: epriestley.

I rolled this into D13122.

This revision now requires changes to proceed.Jun 2 2015, 5:28 PM