search

dabapps / django-rest-framework-serialization-spec

DEPRECATED, see https://github.com/dabapps/django-readers instead
MIT License
11 stars 0 forks source link

Colliding M2M fields #58

Closed j4mie closed 4 years ago

j4mie commented 4 years ago

This is an attempt to isolate and fix #57

At the moment, we have a test case which attempts to capture the problem - but it actually passes at the moment, so it mustn't be right. The issue does specify that the problem only appears for "id-list fields", but I've also tried with id instead of name and the functionality still seems to work.

j4mie commented 4 years ago

The test case now appears to capture the problem.

j4mie commented 4 years ago

Turned out to be an issue with closure scoping.

j4mie commented 4 years ago

👍