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

Add reverse fk handling in the default serializer. #38

Open
wants to merge 11 commits into
base: master
Choose a base branch
from

Conversation

lauxley
Copy link
Contributor

@lauxley lauxley commented Feb 4, 2016

No description provided.

@lauxley
Copy link
Contributor Author

lauxley commented Feb 4, 2016

  • test reverse m2M
  • mappings
  • update docs (how to use a reverse relationship - not picked up by default by the serializer)

@Atala
Copy link
Contributor

Atala commented Feb 26, 2016

Do you plan to merge it soon? That would be nice.

@lauxley
Copy link
Contributor Author

lauxley commented Feb 26, 2016

The only thing that still prevents me from merging is the RelatedManager on ManyToMany fields that complains when i try to fake the relationship on deserialization. I'm not sure what would be the best approach to the problem so i pushed my wip so far if someone has an idea and wants to look at it.
The problem arises in serializers.EsJsonToModelMixin.instanciate()

maybe a custom manager on the related field would work ...

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

Successfully merging this pull request may close these issues.

2 participants