Skip to content

Conversation

@Ivaylo-Bachvarov
Copy link
Contributor

Take a look at this.

We added:

  • An example for nested serializers in the demo app.
  • Support for nested serializer in nested serializer (recursive)
  • User friendly interface for the nested serializer data.

Things that we are going to add in a separate pull request:

  • Support for Nested Serialization in Live API Endpoints.

Authors: Me and @Ivo-Donchev 🐼 🍻

@kdkocev
Copy link

kdkocev commented Jun 22, 2016

👍

2 similar comments
@Ivo-Donchev
Copy link

👍

@Rositsazz
Copy link

👍

@manosim
Copy link
Owner

manosim commented Jun 23, 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?

@Ivaylo-Bachvarov
Copy link
Contributor Author

Ivaylo-Bachvarov commented Jun 23, 2016

@ekonstantinidis

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 🍻

@Ivaylo-Bachvarov
Copy link
Contributor Author

@ekonstantinidis The PR is up to date with master now :)

@manosim
Copy link
Owner

manosim commented Jun 24, 2016

Awesome! It would be great if you can add some tests for this PR at some point?

@Ivaylo-Bachvarov
Copy link
Contributor Author

@ekonstantinidis Upcoming commits with tests and Live API support in this pull request ;)
@Ivo-Donchev will commit soon 🍻

@Ivaylo-Bachvarov
Copy link
Contributor Author

@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.

@manosim
Copy link
Owner

manosim commented Jun 30, 2016

Hello @Ivaylo-Bachvarov! Sorry for the late reply - been really busy. That looks great so let's merge it!

@manosim manosim merged commit 875a600 into manosim:master Jun 30, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment