smartsheet-platform / smartsheet-csharp-sdk

Library that uses C# to connect to Smartsheet services.
Apache License 2.0
35 stars 30 forks source link

Fixed double url escaping for the search api call #116

Closed thijs4ef closed 4 years ago

thijs4ef commented 4 years ago

This pull request solves the double url escaping problem in the sdk for the /search api call. For example: in the current sdk version a space will turn into %2520, it should be %20.

Escaping for the search call is done in main/Smartsheet/Api/Internal/SearchResourcesImpl.cs and in smartsheet-csharp-sdk/main/Smartsheet/Api/Internal/Util/QueryUtil.cs in the current sdk.

timwellswa commented 4 years ago

+1 Thanks for this!