New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Make NuGet OData search case-insensitive #1775

Merged
merged 1 commit into from Jun 30, 2016

Conversation

Projects
None yet
2 participants
@trondd
Contributor

trondd commented Jun 30, 2016

The original code would use the package name as the search argument, which casued the following URL to return no packages:

https://www.nuget.org/api/v2/Packages?$filter=Id%20eq%20%27Log4Net%27

Make NuGet OData search case-insensitive
The original code would use the package name as the search argument, which casued the following URL to return no packages:

https://www.nuget.org/api/v2/Packages?$filter=Id%20eq%20%27Log4Net%27

@forki forki merged commit fd30358 into fsprojects:master Jun 30, 2016

1 of 2 checks passed

continuous-integration/travis-ci/pr The Travis CI build failed
Details
continuous-integration/appveyor/pr AppVeyor build succeeded
Details
@forki

This comment has been minimized.

Show comment
Hide comment
@forki

forki Jun 30, 2016

Member

thanks

Member

forki commented Jun 30, 2016

thanks

@trondd

This comment has been minimized.

Show comment
Hide comment
@trondd

trondd Jun 30, 2016

Contributor

Did you notice that the Mono build was red?

Contributor

trondd commented Jun 30, 2016

Did you notice that the Mono build was red?

@forki

This comment has been minimized.

Show comment
Hide comment
@forki

forki Jun 30, 2016

Member

Yes. That's unrelated, right? Need to investigate tomorrow.
On Jun 30, 2016 6:58 PM, "Trond Danielsen" notifications@github.com wrote:

Did you notice that the Mono build was red?


You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
#1775 (comment),
or mute the thread
https://github.com/notifications/unsubscribe/AADgNAucTFD0k7s-94NuJzzmLv7sKxMfks5qQ_WugaJpZM4JB6Tp
.

Member

forki commented Jun 30, 2016

Yes. That's unrelated, right? Need to investigate tomorrow.
On Jun 30, 2016 6:58 PM, "Trond Danielsen" notifications@github.com wrote:

Did you notice that the Mono build was red?


You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
#1775 (comment),
or mute the thread
https://github.com/notifications/unsubscribe/AADgNAucTFD0k7s-94NuJzzmLv7sKxMfks5qQ_WugaJpZM4JB6Tp
.

@izhovkin izhovkin referenced this pull request Nov 17, 2016

Merged

Added missing tolower() #2023

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment