search

DevLyon / mixter

CQRS and Event Sourcing Koans
http://devlyon.fr/mixter
MIT License
311 stars 81 forks source link

[Timeline] Update timeline when receiving a FolloweeMessagePublished #2

Closed jeantil closed 9 years ago

jeantil commented 9 years ago

Est-ce que ça vous choque comme implémentation ? ça me parait plus simple que tout ce qu'on a fait pour l'instant.

fpellet commented 9 years ago

Non, cela revient au même entre utiliser le repo message ou timeline. Cela semble plus simple. Mais peut être l'avantage d'utiliser le repository message, c'est de fusionner deux sources de données pour alimenter la projection. Mais bon, si ils arrivent jusque là, c'est déjà très bien :)

jeantil commented 9 years ago

bon du coup je garde cette version là pour le java si c'est une connerie je referai le truc

devcrafting commented 9 years ago

Mouef, mettre des getters sur les agrégats, c'est pas top (mais à la limite mieux encapsulé que de lire directement l'EventStore comme au début)...je serais vraiment plus pour mettre les données nécessaires dans l'événement. Ou d'utiliser un repository de projections (cf. mail).

jeantil commented 9 years ago

euh je ne te suis pas clement, ou alors ton commentaire n'est pas en lien avec la pull request ... il n'y a pas de getter sur l'agregat en tout cas pas dans ce commit