-
Notifications
You must be signed in to change notification settings - Fork 213
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fix slot depth unit to 'slot'. Add block type #139
Conversation
specifications/api/swagger.yaml
Outdated
@@ -35,12 +35,27 @@ numberOfSlots: &numberOfSlots | |||
properties: | |||
quantity: | |||
type: integer | |||
minimum: 0 | |||
minimum: 1337 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
a bit more "arbitrary" number
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hmm... But this is wrong here, it states that the minimum should be 1337 😶
Did you mean to add it as example
?
unit: | ||
type: string | ||
enum: | ||
- slot | ||
example: 1337 | ||
example: "slot" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
we already discussed over slack that old slot unit was probably a mistake
6a0871e
to
eb908bb
Compare
bd3862b
to
e313170
Compare
Issue Number
#93
Overview
Comments
This adds slight modification to our
swagger.json
to make types more strict. It might be overkill though