Skip to content
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

Odata string key #284

Closed
wants to merge 2 commits into from
Closed

Conversation

SDEBEUL
Copy link
Contributor

@SDEBEUL SDEBEUL commented May 28, 2021

Hi,

I would like to merge this commit.
I'm not that confident if this is 100% correct.
But please be so kind to provide advice if I'm wrong.

I have an odata endpoint.
Microsoft.AspNetCore.OData Version="7.5.6"
Where the key is not an int but a string.

the key fails do bind / find the route when i just call.
Odata\Client(the client name)
I need to call it with ' around the keyname
Odata\Client('the client name')

This was not supported in the GetUrl method.

I made a path for this.
Not sure if this is the best / cleanest solution.

Hope you want to merge this :)

br

odata keys can be strings.
wheb an odata key is a string the url needs ' delimiters around the keys
gustavnavar added a commit that referenced this pull request Jun 2, 2021
@gustavnavar
Copy link
Owner

I've implemented another code for string keys in GridBlazor 2.1.15.

Please, let me know if it works for you and I will close this pull request

rgentry09 pushed a commit to rgentry09/ERP-ASP.NET that referenced this pull request Jul 10, 2023
dev0926 added a commit to dev0926/gridNET that referenced this pull request Feb 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants