-
Notifications
You must be signed in to change notification settings - Fork 209
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
Handle batch calls #677
Handle batch calls #677
Conversation
Hi @m4gus88, thanks for contributing! Can you add some tests? |
@fsouza I don't 'really know anything about Go testing, but I'll give it a try |
Add a rudamental test for batch delete
13afd30
to
28cb47c
Compare
@fsouza Any chance this will be merged and released any time soon? |
Hey, somehow I missed this after the last approval for running, I'm really sorry about that. I'll review it today and if everything looks good I'll merge it and release it! |
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.
Looks good! Just some minor points, but I'll merge it once they're addressed!
Thank you! |
Addressing issue #411, added a handler for batch calls.