Page MenuHomePhabricator

Exclude disabled (disapproved) users from count on People application on homepage
ClosedPublic

Authored by epriestley on Mar 10 2014, 10:20 PM.
Tags
None
Referenced Files
F14066363: D8486.diff
Tue, Nov 19, 9:37 AM
F14051451: D8486.diff
Fri, Nov 15, 1:04 AM
F14047945: D8486.id20102.diff
Thu, Nov 14, 5:55 AM
F14041832: D8486.id20101.diff
Mon, Nov 11, 9:58 PM
F14037717: D8486.diff
Sun, Nov 10, 6:35 PM
F14036879: D8486.id20102.diff
Sun, Nov 10, 12:43 PM
F14024418: D8486.diff
Thu, Nov 7, 9:19 AM
F13978898: D8486.id20101.diff
Oct 19 2024, 1:33 AM
Subscribers

Details

Summary

The People application shows users awaiting approval, but incorrectly counts disabled users (i.e., users who were not approved).

Instead, count only non-disabled, non-approved users.

Test Plan

My homepage count dropped from 4 to 1, corresponding to the actual number of accounts.

Diff Detail

Repository
rP Phabricator
Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

btrahan edited edge metadata.
This revision is now accepted and ready to land.Mar 10 2014, 10:26 PM
epriestley updated this revision to Diff 20102.

Closed by commit rP0a779b60a285 (authored by @epriestley).