expose collection cannot specify a :serializer

[hlxwell]

If I wants to expose a array of objects, each one can use a serializer, but I didn't find any :each_serializer option there.

What I should do?

I made a pull request, it works for me, but I am not sure if it is the correct solution.

#106

[mattcl]

I would also like to see this addressed

[Sutto]

Having a look now, I'll tweak it slightly and see if this can fix it - It's also an issue (I believe) with paginated collections.

[Sutto]

Oh, and the short way - If you specify :each_serializer, also specify :serializer as the ArraySerializer in whatever AMS version you're using, if you're using AMS.

[hlxwell]

actually my requirement was very simple, so I didn't find the correct way to do it.

I just added below code to all model class then solved. doesn't need each_serializer

  def active_model_serializer
    "#{self.class.name}Serializer".constantize
  end

[Sutto]

I'm working on support for this at the moment - there are a few edge cases we need to keep in mind when doing so & hence I figured I'd update here:

1. We have to use a check other than #to_a for 'collections' as hashes implement it, when we want to treat them as singular.
2. We should respect if someone sets :serializer and :each_serializer, only using the first / not doing anything special.
3. We need to support: Arrays, Sets and Scopes at a minimum.

This'll be pushed out in the newest release, apologies for the massive delays.

[hlxwell]

@Sutto Agree with your comments.

[DmitryKK]

@Sutto is there any progress?

[Sutto]

@DmitryKK there hadn't been, but going to do this for 1.12 in the next few days

[Sutto]

Try the version in master - I just pushed rough support for this.

[DmitryKK]

Thanks, i tried. Serializer works only for the first record

[Sutto]

Have you got a specific example of what you're doing so I can try again?

On Tue, May 19, 2015 at 8:52 AM -0700, "Dmitry Krakosevich" notifications@github.com wrote:

Thanks, i tried. Serializer works only for the first record

—
Reply to this email directly or view it on GitHub.

[Sutto]

All good - Fixed and pushed