swagger-express-middleware icon indicating copy to clipboard operation
swagger-express-middleware copied to clipboard

Published 20 hours ago • verified publisher icon APIDevTools

Switching to Z-Schema

Open Santinell opened this issue 9 years ago • 5 comments

  • 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

Santinell avatar Jan 21 '16 12:01 Santinell

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.

JamesMessinger avatar Jan 21 '16 15:01 JamesMessinger

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

winniehell avatar Jan 22 '16 17:01 winniehell

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?

Santinell avatar Mar 03 '16 06:03 Santinell

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

JamesMessinger avatar Mar 03 '16 06:03 JamesMessinger

Really? a couple weeks?

Santinell avatar May 26 '16 12:05 Santinell