Skip to content

Commit 37f41fe

Browse files
authored
Merge pull request #1438 from json-api-dotnet/openapi-fix-id-parameter-type
OpenAPI: Fix "id" parameter in URL to be string
2 parents fda2b36 + b0fa688 commit 37f41fe

File tree

26 files changed

+373
-603
lines changed

26 files changed

+373
-603
lines changed

docs/usage/openapi-client.md

+1-1
Original file line numberDiff line numberDiff line change
@@ -94,7 +94,7 @@ The next steps describe how to generate a JSON:API client library and use our pa
9494
person => person.FirstName))
9595
{
9696
// Workaround for https://github.com/RicoSuter/NSwag/issues/2499.
97-
await TranslateAsync(async () => await apiClient.PatchPersonAsync(1, null, patchRequest));
97+
await TranslateAsync(async () => await apiClient.PatchPersonAsync(patchRequest.Data.Id, null, patchRequest));
9898

9999
// The sent request looks like this:
100100
// {

src/Examples/JsonApiDotNetCoreExample/Properties/launchSettings.json

+2-2
Original file line numberDiff line numberDiff line change
@@ -12,15 +12,15 @@
1212
"IIS Express": {
1313
"commandName": "IISExpress",
1414
"launchBrowser": true,
15-
"launchUrl": "api/todoItems?include=owner,assignee,tags&filter=equals(priority,'High')",
15+
"launchUrl": "swagger",
1616
"environmentVariables": {
1717
"ASPNETCORE_ENVIRONMENT": "Development"
1818
}
1919
},
2020
"Kestrel": {
2121
"commandName": "Project",
2222
"launchBrowser": true,
23-
"launchUrl": "api/todoItems?include=owner,assignee,tags&filter=equals(priority,'High')",
23+
"launchUrl": "swagger",
2424
"applicationUrl": "https://localhost:44340;http://localhost:14140",
2525
"environmentVariables": {
2626
"ASPNETCORE_ENVIRONMENT": "Development"

0 commit comments

Comments
 (0)