Skip to content

Serialize OpenApiDate values properly to short date - #1102

Merged
Irvine Sunday (irvinesunday) merged 2 commits into
vnextfrom
is/fix-date-serialization
Dec 8, 2022
Merged

Serialize OpenApiDate values properly to short date#1102
Irvine Sunday (irvinesunday) merged 2 commits into
vnextfrom
is/fix-date-serialization

Conversation

@irvinesunday

Copy link
Copy Markdown
Contributor

Fixes#1101

This PR:

  • Serializes OpenApiDate values to short date.
  • Updates test to validate the above.

Using value 01/01/0001 could be represented as 1/1/0001 in a machine using a different regional format, and thus cause the test to fail
@sonarqubecloud

Copy link
Copy Markdown

@irvinesunday
Irvine Sunday (irvinesunday) deleted the is/fix-date-serialization branch December 8, 2022 12:03
AfsalMcQueen (Afsalmc) pushed a commit to Afsalmc/OpenAPI.NET that referenced this pull request Dec 14, 2022
* Serialize OpenApiDate properly as Date
* Change test date value
Using value 01/01/0001 could be represented as 1/1/0001 in a machine using a different regional format, and thus cause the test to fail
Sign up for freeto 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.

OpenApiDate values are serialized as DateTime with timestamps

3 participants

@irvinesunday@baywet@millicentachieng