bmihelac / ra-data-django-rest-framework

react-admin data provider for Django REST framework
MIT License
100 stars 28 forks source link

Cannot read properties of undefined (reading 'id') after deleteMany #49

Closed e-stepanov closed 2 years ago

e-stepanov commented 2 years ago

Handling of the server response in the deleteMany method expects json with id field but django rest framework returns response without body.

According to react-admin docs the ids of the deleted records are optional in deleteMany. So that an empty array can be returned as a data value.

I can create a pull request if you confirm this bug.

bmihelac commented 2 years ago

@e-stepanov thanks for this issue report. I am currently involved in other projects, and cannot devote time to ra-data-django-rest-framework but would be glad to merge a pull request.

e-stepanov commented 2 years ago

Done! Will you release a new version with this fix?

bmihelac commented 2 years ago

@e-stepanov thanks for PR, it is in 0.2.1