-
Notifications
You must be signed in to change notification settings - Fork 32
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
Add ToDictionary() and other things #19
Conversation
Hi, I looked at the changes you want to make. In general, I support, but I want to make changes, taking into account how I see the use of current changes and the use of the library in the future I wanted to publish a new branch and do PR in https://github.com/alexandresanlim/OData.QueryBuilder but I did not have enough rights :) |
Nice! I invited you as a contributor, I think that solves the issue of access. |
I created a PR. If it meets all your requirements, then I propose to merge :) https://github.com/alexandresanlim/OData.QueryBuilder/pull/1 |
…-opportunities #ZEXSM#review support new opportunities
Merge confirmed. |
…-opportunities up coverage
Add support ToDictionary():
In some cases it is necessary to send a dictionary to url.
eg:
var dic = odataBuilder.ToDicionary();
The result is:
Send url with dicionary:
return await ServiceClient.InvokeApiAsync<List<Core.Data.DataModels.Booking>>("Booking/GetAllByDataStoreId", HttpMethod.Get, **dic**);
This modification does not change the ToUrl() :)
Add support Contains() text:
eg:
Url result:
http://mock/odata/ODataType?$filter=substringof('0',toupper(ODataKind/ODataCode/Code)) or substringof('55',toupper(ODataKindNew/ODataCode/Code))
Add support enum ToString() compare:
eg:
Url result:
http://mock/odata/ODataType?$filter=Color eq 'Blue'&$skip=1&$top=10