Support parsing schema values exceeding decimal ranges by replacing decimal.Parse with double.Parse #1594
Conversation
…al's range; parse exclusivemax and min values as boolean types
…ilder and extension methods for fetching their values from the corresponding JSON schema
MaggieKimani1
requested review from
andrueastman,
baywet,
darrelmiller,
irvinesunday,
millicentachieng,
peombwa and
zengin
as code owners
|
|
|
||
| // multipleOf | ||
| writer.WriteProperty(OpenApiConstants.MultipleOf, schema.GetMultipleOf()); | ||
| writer.WriteProperty(OpenApiConstants.MultipleOf, schema.GetOpenApiMultipleOf()); |
There was a problem hiding this comment.
Are we forced to prefix with OpenAPI to differentiate from the jsonschema library methods?
Is there any reason why we can't use that library's methods?
There was a problem hiding this comment.
Yes prefixing with OpenApi helps to distinguish our custom implementation from the JsonSchema.NET library's.
Also we're overriding the library's methods because the keywords such as MultipleOf, Maximum and Minimum are defined as decimal types and we need to replace that and use double for us to support parsing schema values exceeding decimal's ranges.
There was a problem hiding this comment.
Wouldn't it be better to PR upstream instead of maintaining our own code in this case?
|
sorry about the closure, we have aligned branching with other repositories, if this is still relevant, please re-open on main |
Fixes #1106,
fixes #1265