-
Notifications
You must be signed in to change notification settings - Fork 31
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
ArrayRef[HashRef] param #71
Comments
[[[egg on face]]] We were using old release, seems there is better behaviour in latest release. will experiment with that a bit first |
I can see the very same behaviour on current master. |
I assume the issue is in https://github.com/khrt/Raisin/blob/master/lib/Raisin/Plugin/Swagger.pm#L286 where everything which is not |
It would be great if we could still use `group` syntax to describe the
hashref that's in arrayref.
…On Sun., Jul. 28, 2019, 09:18 Artur Khabibullin, ***@***.***> wrote:
I assume the issue is in
https://github.com/khrt/Raisin/blob/master/lib/Raisin/Plugin/Swagger.pm#L286
where everything which is not HashRef is ignored.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#71?email_source=notifications&email_token=AABI6XAZZHYI7H6ZQSBESHTQBXBGZA5CNFSM4IHHE3R2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD27BYNY#issuecomment-515775543>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AABI6XCDA3GXZGZDAR5UBD3QBXBGZANCNFSM4IHHE3RQ>
.
|
# Please enter the commit message for your changes. Lines starting # with '#' will be kept; you may remove them yourself if you want to. # An empty message aborts the commit. # # Date: Sun Jul 28 20:34:28 2019 +0200 # # On branch array-of # Changes to be committed: # modified: lib/Raisin/Plugin/Swagger.pm # # Untracked files: # examples/test.pl #
* generate any-type items for ArrayRef w/o `using` (#71); * fix `tags` object of spec to be compliant;
I've kinda fixed it. The fix is that if type of a parameter is Not sure if it's what you want, but at least the spec is valid now. |
Released to CPAN to anyway, even if it doesn't solve your issue. |
Thanks for such quick action, this helps! I'll keep the ticket open in hopes we can get more extensive |
We would like an endpoint to to take a list of objects as an input, ex:
Naive approach, where we don't validate each object in list would be like this
This generates following bit in swagger:
But there is no definition of
Entries-MD5Hash
indefinitions
section in swagger, which makes the swagger.json out of spec.Ideally, we would be able to define the structure of the HashRef too as part of the definition.
Any suggestions how to deal with this?
Thanks!
The text was updated successfully, but these errors were encountered: