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
Unknown Object (File)
Wed, Sep 11, 8:44 AM
Unknown Object (File)
Sat, Aug 31, 2:18 PM
Unknown Object (File)
Tue, Aug 27, 9:26 PM
Unknown Object (File)
Fri, Aug 23, 11:59 PM
Unknown Object (File)
Fri, Aug 23, 2:10 PM
Unknown Object (File)
Thu, Aug 22, 4:06 AM
Unknown Object (File)
Aug 17 2024, 6:36 AM
Unknown Object (File)
Aug 17 2024, 1:21 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).