dotnet / docs

This repository contains .NET Documentation.
https://learn.microsoft.com/dotnet
Creative Commons Attribution 4.0 International
4.24k stars 5.88k forks source link

Aggregate in domain event example #19367

Open amartinkyan opened 4 years ago

amartinkyan commented 4 years ago

As stated in the beginning of the page: use domain events to explicitly implement side effects across multiple aggregates and also the purpose of domain events is to notify that something has already happened. I believe that use domain entity Order in 'Implement domain events' section isn't appropriate because in this case one can call methods on the entity and produce even more side effects for this aggregate. Of course as usual 'it depends' but I think it worth to mention consequences of such approach.


Document Details

Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

sughosneo commented 4 years ago

Hi @amartinkyan, thanks for the feedback!

Would you like to submit a PR to the docs repo with your proposed changes ?