-
Notifications
You must be signed in to change notification settings - Fork 16
Allow array query params #41
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
Allow array query params #41
Conversation
@proofrock I will rebase this after #38 is merged. |
Looks nice, thanks! Yes, let's rebase it after #38 |
Thanks. There are still a couple of errors and conflicts, would you solve them (whenever you can)? Please try a "make test" afterwards.
|
Good code anyway, thanks! 👍 |
Oh yep I'm trying to rebase, but it seems simpler to create a new branch and copy the changes manually anyway 😅. I'll let you know when the new branch is ready |
9d6b6d3
to
8115601
Compare
@proofrock it should be ready |
Many thanks!! |
Next step is to write documentation for all that, but before I'd like to do the resultset format "change" |
Resolve #40.
Sample
curl
:Sample body:
Sample response: