Skip to content
This repository has been archived by the owner on Mar 6, 2024. It is now read-only.

Switching to Z-Schema #31

Closed
wants to merge 4 commits into from
Closed

Conversation

Santinell
Copy link

  • removed tests with using strings and dates in fields minimum/maximum
  • changed mechanism of "no required" (from "schema.required = [];" to "delete schema.required;")
  • delete schema.required if we copy schema from param and if required is boolean

@JamesMessinger
Copy link
Member

Hi. Thanks for the PR. It's much appreciated, and I'll definitely take a look at it as soon as I get a chance. It may be a few days though, since things are super busy for me right now.

@winniehell
Copy link
Contributor

This pull request includes the changes of #26. Maybe that can be split up?

@Santinell
Copy link
Author

BigstickCarpet, where are you? We are miss you!

Looks like z-schema is not planning support of json-schema v5 in the near future.
May be better switch to ajv?

@JamesMessinger
Copy link
Member

I'm in India on a business trip. working around the clock. no time to work on code right now. But I'll get back to it when I return to the U.S. in a couple weeks

@Santinell
Copy link
Author

Really? a couple weeks?

@APIDevTools APIDevTools deleted a comment Aug 24, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants