Skip to content

Allows to override date-time and duration validators#787

Merged
stevehu merged 1 commit intonetworknt:masterfrom
josejulio:fixes-784
May 26, 2023
Merged

Allows to override date-time and duration validators#787
stevehu merged 1 commit intonetworknt:masterfrom
josejulio:fixes-784

Conversation

@josejulio
Copy link
Copy Markdown
Contributor

Fixes #784

@stevehu stevehu requested a review from fdutton May 25, 2023 21:41
Copy link
Copy Markdown
Contributor

@fdutton fdutton left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me

MikelAlejoBR pushed a commit to MikelAlejoBR/event-schemas-java that referenced this pull request May 26, 2023
Josejulio realized that there is an issue with the
"json-schema-validator"[1] dependency we depend on. Basically the custom
formatters are not being honored, and that causes some date times to not
be read as expected.

Josejulio is already working on a fix[2], but in the meantime we will
downgrade the library we depend on to a previous version.

[1]: networknt/json-schema-validator#784
[2]: networknt/json-schema-validator#787
josejulio pushed a commit to RedHatInsights/event-schemas-java that referenced this pull request May 26, 2023
Josejulio realized that there is an issue with the
"json-schema-validator"[1] dependency we depend on. Basically the custom
formatters are not being honored, and that causes some date times to not
be read as expected.

Josejulio is already working on a fix[2], but in the meantime we will
downgrade the library we depend on to a previous version.

[1]: networknt/json-schema-validator#784
[2]: networknt/json-schema-validator#787

Co-authored-by: Mikel Alejo <>
@stevehu stevehu merged commit aa9f164 into networknt:master May 26, 2023
@josejulio josejulio deleted the fixes-784 branch May 26, 2023 16:44
@josejulio
Copy link
Copy Markdown
Contributor Author

thanks @stevehu and @fdutton

Do you have any idea of when the next release will happen?
No pressure, just to plan accordingly 😄

@stevehu
Copy link
Copy Markdown
Contributor

stevehu commented May 27, 2023

@josejulio I have released 1.0.83, and it will fix the issue. Thanks a lot for your help.

@josejulio
Copy link
Copy Markdown
Contributor Author

@josejulio I have released 1.0.83, and it will fix the issue. Thanks a lot for your help.

awesome, thank you!

@kad-koninr1
Copy link
Copy Markdown

Thank you @josejulio and @stevehu!

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.

Custom date-time format not possible since version 1.0.82

4 participants