-
Notifications
You must be signed in to change notification settings - Fork 183
Nested serializers support #104
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 serializers support #104
Conversation
kdkocev
commented
Jun 22, 2016
👍
2 similar comments
Ivo-Donchev
commented
Jun 22, 2016
👍
Rositsazz
commented
Jun 22, 2016
👍
Hello everyone! Thank you for the PR -looks really good! Can you bring it up to date with master so that we can merge it?
Today I will bring it up to date and I will ping you here for a merge.
We are preparing some more PR for that great project 🍻
@ekonstantinidis The PR is up to date with master now :)
Awesome! It would be great if you can add some tests for this PR at some point?
@ekonstantinidis Upcoming commits with tests and Live API support in this pull request ;)
@Ivo-Donchev will commit soon 🍻
@ekonstantinidis I think that we are done with this.
We added support for many=True in nested serializing.
This PR is ready for merge.
Another PR is comming with Live API Support.
Hello @Ivaylo-Bachvarov! Sorry for the late reply - been really busy. That looks great so let's merge it!
Take a look at this.
We added:
Things that we are going to add in a separate pull request:
Authors: Me and @Ivo-Donchev 🐼 🍻