Skip to content

Allow to define custom serializer for given class - #813

Merged
kurko merged 1 commit into
rails-api:0-9-stablefrom
jtomaszewski:feature/custom-serializer-class
Mar 9, 2015
Merged

Allow to define custom serializer for given class#813
kurko merged 1 commit into
rails-api:0-9-stablefrom
jtomaszewski:feature/custom-serializer-class

Conversation

@jtomaszewski

Copy link
Copy Markdown

.. by defining #serializer_class method in serialized object's class. Resolves #515.

This is a really small fix and has a test - so I don't see any reasons why it shouldn't be merged.

@joshsmith

Copy link
Copy Markdown
Contributor

I say 👍

Comment thread lib/active_model/serializer.rb Outdated

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We're using 2 spaces indentation.

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fixed (force pushed a reworked commit)

.. by defining #serializer_class method in serialized object's class. Resolves rails-api#515.
@jtomaszewski
jtomaszewski force-pushed the feature/custom-serializer-class branch from 8e6a0e8 to d8b78a3 Compare March 8, 2015 13:59
kurko added a commit that referenced this pull request Mar 9, 2015
…lass

Allow to define custom serializer for given class
@kurko
kurko merged commit 55210ab into rails-api:0-9-stable Mar 9, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants