Skip to content
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

Implement support for bulk delete operation #1242

Closed
wants to merge 1 commit into from

Conversation

fkirill
Copy link

@fkirill fkirill commented Feb 3, 2019

Thank you for submitting a pull request!

Please make sure you have signed our Contributor License Agreement (CLA).
We are not asking you to assign copyright to us, but to give us the right to distribute your code without restriction. We ask this of all contributors in order to assure our users of the origin and continuing existence of the code.
You only need to sign the CLA once.

Copy link
Member

@jbaiera jbaiera left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@fkirill Thanks for opening a PR for this, could you add some tests to make sure that the correct bulk payloads are being created as well as an integration test to make sure that it works for all the integrations?

@jbaiera
Copy link
Member

jbaiera commented Aug 23, 2019

Closing this in favor of #1324

@jbaiera jbaiera closed this Aug 23, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants