jorge07 / symfony-6-es-cqrs-boilerplate

Symfony 6 DDD ES CQRS backend boilerplate.
MIT License
1.07k stars 187 forks source link

UserView::serialize | created_at field #175

Closed abelardolg closed 4 years ago

abelardolg commented 4 years ago

Hi there,

1: I think it's a typo at the line 25 of this class.

2: inside the serialize method the "created_at" and "updated_at" fields are not returning inside the array : is it ok?

Best regards.

jorge07 commented 4 years ago

Hi @abelardolg

1 - True, will submit a patch

2 - True, it depends how much data you want to expose.

I also rethinking about the read model to avoid serialisation as much as possible to expect modifications in this side

thanks

jorge07 commented 4 years ago

Closed by mistake. I'm working on a fix for this, plus some optimization in the serializer side

jorge07 commented 4 years ago

184 fixed in