Paging seems broken for Milestone.GetForRepository about octokit.net HOT 6 CLOSED

It's to do with the ApplyParameters extension we use. Here's a failing test:

[Fact]
public void DoesNotDropPageFromQueryStringWhenUsingParameters()
{
    var uri = new Uri("https://api.github.com/repositories/1/milestones?state=closed&sort=due_date&direction=asc&page=2");

    var parameters = new Dictionary<string, string> { { "state", "open" }, { "sort", "other"} };

    var actual = uri.ApplyParameters(parameters);

    Assert.True(actual.Query.Contains("page=2"));
}

cc @haacked if he wants to look into this fix

from octokit.net.

I've started a PR for this fix here

https://github.com/octokit/octokit.net/tree/milestones-kill-your-rate-limit

I haven't confirmed the milestones client works as expected, but that's easy to chase down now...

from octokit.net.

@haacked @shiftkey i notice 0.1.3 is out. did this one make it into the release?

from octokit.net.

https://github.com/octokit/octokit.net/releases/tag/v0.1.3

"Fixed bug in applying query parameters that could cause paging to continually request the same page"

looks like it. so shall we close this?

from octokit.net.

@SimonCropp Can you try it out?

from octokit.net.

Works on my machine

from octokit.net.