-
Notifications
You must be signed in to change notification settings - Fork 53
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
support array response schemas #20
base: main
Are you sure you want to change the base?
Conversation
Anyone there to review this? I would really like to use the official package rather than my local version. |
Array support is very much needed! |
@SaltyAom sry for ping, but have you considered this? We need array support! |
hi @david-plugge @BE-CH ! I would love to review this, there is some conflicts in i'm a new maintainer helping out! |
i realize this might be a hard conflict to resolve since the new release, let me know if I can help at all! |
Awesome, i will resolve the issues as soon as possible :) |
@david-plugge Can I finish this pr for you? |
Feel free to do so :) |
hey btw #179 would also fix this |
closes #8
I don´t really understand why only
type
,properties
andrequired
we´re allowed, please elaborate on that if it´s important.