Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

Improve serialization example #2402

Closed
wants to merge 1 commit into
from

Conversation

Projects
None yet
5 participants
Contributor

tvlooy commented Mar 30, 2013

Improve serialization example

Q A
Doc fix? yes
New docs? no
Applies to all
Fixed tickets #2225
Improve serialization example
Improve serialization example

| Q             | A
| ------------- | ---
| Doc fix?      | yes
| New docs?     | yes
| Applies to    | all
| Fixed tickets | #2225

@tvlooy tvlooy referenced this pull request in schmittjoh/serializer Mar 30, 2013

Merged

Improve serialization example #71

Member

stof commented Mar 30, 2013

new docs: yes ? this is not adding new doc but only fixing existing one.

Contributor

tvlooy commented Mar 30, 2013

I saw it as an extension and thereby something new. Changed now.

Member

wouterj commented Mar 30, 2013

It doesn't make much sense. I also think it's a new docs.

Contributor

tvlooy commented Mar 30, 2013

Well, does it matter?

Contributor

Sgoettschkes commented Mar 30, 2013

👍

Member

weaverryan commented Mar 31, 2013

Awesome job Tom - another PR merged, issue closed.

I've patched this into the 2.0 branch at sha: 1a8d0dc

Thanks!

@weaverryan weaverryan closed this Mar 31, 2013

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment