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

Nested writable ListFields break the browsable API #3500

Closed
jackmccloy opened this issue Oct 14, 2015 · 1 comment
Closed

Nested writable ListFields break the browsable API #3500

jackmccloy opened this issue Oct 14, 2015 · 1 comment

Comments

@jackmccloy
Copy link

When you do a PUT request via the browsable API on an endpoint that has a nested writable ListField in it's serializer, it wipes out any existing information in this list field. Since lists are not currently supported in HTML input, as the browsable API states, it seems to me that they should be ignored by the serializer's update method, as opposed to passing an empty list. This way a user could still use the browsable api to update the other non-ListFields without unexpectedly losing data. Thoughts?

Happy to work on a fix for this one over the weekend if it'd help. In the 3.0 release notes, it was mentioned that nested lists were planned for 3.1, and if/when nested lists are supported it'll render this issue moot.

@xordoquy
Copy link
Collaborator

Duplicate of #2148

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

No branches or pull requests

2 participants