Homec4science

Allow Almanac interfaces to be browsed

Authored by epriestley <git@epriestley.com> on Feb 21 2016, 12:12.

Description

Allow Almanac interfaces to be browsed

Summary:
Fixes T10205. Ref T10246. Previously, the issue was that the result set was not ordered, so "More Results" would not have been able to work in a reasonable way if there were more than 100 matching interfaces.

You would have seen 100 interfaces more or less at random, then clicked "more" and gotten 100 more random interfaces.

Now, you would see 100 "a" interfaces, then click more to get the next 100 alphabetical interfaces (say, "b" and "c" interfaces).

Test Plan:

  • Clicked browse when binding an interface.
  • Got a browse dialog.
  • Artificially set query limit to 1, paged through "local" interfaces in an ordered, consistent way.

{F1121313}

Reviewers: chad

Reviewed By: chad

Maniphest Tasks: T10205, T10246

Differential Revision: https://secure.phabricator.com/D15320

Details

Committed
epriestley <git@epriestley.com>Feb 21 2016, 19:45
Pushed
aubortJan 31 2017, 17:16
Parents
rPH1b6ddae6b2d8: Allow Almanac devices to be queried and sorted by name
Branches
Unknown
Tags
Unknown

Event Timeline

epriestley <git@epriestley.com> committed rPH22d2b7ed0414: Allow Almanac interfaces to be browsed (authored by epriestley <git@epriestley.com>).Feb 21 2016, 19:45