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

plans for batch actions #289

Closed
rall opened this issue Feb 4, 2015 · 5 comments
Closed

plans for batch actions #289

rall opened this issue Feb 4, 2015 · 5 comments
Labels

Comments

@rall
Copy link

rall commented Feb 4, 2015

Are there any plans to allow batch updates and deletes?

@fzaninotto
Copy link
Member

A few random thoughts.

This is one of the less standardized parts of REST. What do you believe is the common REST API for batch update and delete?

Since it's not standardized, the easy way to implement it in ng-admin would be to issue n update or delete requests to the API - DDOSing it, sort of.

So it's easy to do in a bad way, hard to do the right way, and probably hard to implement support on the server-side.

@fzaninotto
Copy link
Member

As a side note, plans for the next major release can be read at the bottom of the 0.5 announcement blog post.

@rall
Copy link
Author

rall commented Feb 4, 2015

ah, thanks for the link. I hadn't seen that.

For the batch operations, my api supports requests like this:

1. PATCH /buttons?attribute=foo { "attribute": "bar" }
2. DELETE /buttons?attribute=qux
  1. update all 'buttons' resources where attribute="foo",
  2. delete all 'buttons' resources where attribute = "qux"

so I was hoping to be able to send requests like that from ng-admin. My api might be an outlier in the world of Rest on this point, so no big deal if you don't want to support these types of request :-)

@fzaninotto
Copy link
Member

We'll do that in the future, our customers need it. The hard part is to understand the good practices on the backend side to support that.

Feel free to implement it for your own needs and send us a PR to see if that can bootstrap the job.

@fzaninotto
Copy link
Member

Implemented in #374

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants