HomePhabricator

Limit fetchUserGenderData to maximum 50 users
0e63dd7f0f0dUnpublished

Unpublished Commit · Learn More

Not On Permanent Ref: This commit is not an ancestor of any permanent ref.
This commit has been deleted in the repository: it is no longer reachable from any branch, tag, or ref.

Description

Limit fetchUserGenderData to maximum 50 users

The 50 is the default. The worst-case scenario here is that users
beyond this limit don't have a gender for a while, until the next call
to the same API endpoint fetches the next 50 users with an unknown
gender.

Doing multiple API calls in advance is not worth it, in my opinion.

Bug: T197858
Change-Id: I0fdcc7ea96a6a5ee3934600c6f0fdc65263276e8

Details

Provenance
thiemowmdeAuthored on Jun 26 2018, 1:15 PM
WMDE-FischCommitted on Jul 3 2018, 10:52 AM
Parents
rERSLcb26b4aa3761: Use cookie to dismiss help dialog and fix failing browser tests
Branches
Unknown
Tags
Unknown
ChangeId
I0fdcc7ea96a6a5ee3934600c6f0fdc65263276e8