Change the API for get_remote_repositories().

Review Request #5903 — Created May 29, 2014 and submitted — Latest diff uploaded

Information

Review Board
master
36034e5...

Reviewers

get_remote_repositories()'s API was using plan IDs as a way of
identifying users vs. organizations, which led to some redundancy.
For example, "private" and "public" for GitHub meant the same thing
(users). GitHub in particular was assuming certain filter types for
repositories, which would be handy to specify from a caller's point of
view.

The function has now been changed to remove the plan parameter and to
instead have a dedicated "owner_type" parameter that specifies what
"owner" refers to. The value is dependent on the hosting service, and
HostingService subclasses are expected to be smart about the defaults.

It also now takes a "filter_type" parameter that allows for filtering
repositories. Possible values are dependent on the hosting service as
well. For instance, GitHub's allowed values consist of what their API
natively allows for their "type=" parameter in repository queries.

Unit tests pass.

Tested with my upcoming web API change, testing user and org lookups
and various filters.

    Loading...