Has_many method 'xxx_ids' about panko_serializer HOT 4 OPEN

@BorisBresciani I can add support for has_many :pankos, embed: :ids in serializer like AMS.

wdyt?

from panko_serializer.

It is a good idea or if not to do as simple attributes without specifying "has_many" like my example

from panko_serializer.

It's simple to do embed because at serializer definition time I don't know which model is serialized in advance so I can't tell if panko_ids is field or method and detecting this at runtime can be a huge performance loss.

I have two options to tackle this:

• Move to strict serializes and don't allow polymorphic serializers, so in addition to attributes, you will specify model Post - and I can tell automatically which method it is, gather type information etc.
• At runtime detect which model I am serializing and cache results of attributes mapping per serializer.

Both solutions will take some time to develop if you can wait two weeks (at max), I will implement them now and in the meanwhile you will do:

attributes :panko_ids
def panko_ids
  object.panko_ids
end

or

attributes :panko_ids
delegate :panko_ids to: :object

from panko_serializer.

Hi,

The second solution seems better

from panko_serializer.