search

miki725 / django-rest-framework-bulk

Django REST Framework bulk CRUD view mixins
Other
521 stars 106 forks source link

Fix bulk updates with unique together #60

Closed sashasimkin closed 1 year ago

sashasimkin commented 7 years ago

This is mostly same approach as in https://github.com/miki725/django-rest-framework-bulk/pull/43 but doesn't do excess database queries. This doesn't cover cases when unique constraint violated within incoming payload. In this case IntegrityError is raised deep inside django.

sashasimkin commented 7 years ago

Also, I see that tests are failing because of reasons not connected to my code. I can fix those, but tell me preferred way where I need to push them. Same PR or different one?