Skip to content
This repository has been archived by the owner on Nov 16, 2023. It is now read-only.

System.UInt32 incorrectly translated to OpenApi format "int32" #227

Open
ps-weber opened this issue Nov 12, 2019 · 1 comment
Open

System.UInt32 incorrectly translated to OpenApi format "int32" #227

ps-weber opened this issue Nov 12, 2019 · 1 comment

Comments

@ps-weber
Copy link

ps-weber commented Nov 12, 2019

A int32 can not fit all possible values of a uint32.
This leads to errors with generated clients.

See TypeExtensions.cs

Basically the same issue as #210, but with uint instead of ushort.

@christopher-bonitz
Copy link

christopher-bonitz commented Dec 6, 2019

Could there be an general cause for these mapping differences?.
I'm encountering something similar between decimal types being explained as doubles.
Less fatal, but it begins to look like a pattern.

Edit: Looking further into the code ad ps-weber referenced, I found that decimal is not supported in the json/openApi spec yet, so my issue can be ignored. I guess double is as good as it gets.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants