search

danielcirket / OpenEventSourcing

MIT License
3 stars 0 forks source link

Split event meta data from the event itself #34

Closed danielcirket closed 3 years ago

AtLeastITry commented 4 years ago

@danielcirket Everything seems to be good as far as i can tell, i especially like that it takes the burden from the consumer to constantly worry about executing the “UpdateFrom(ICommand)” and handling the metadata.

Out of curiosity, what was your reasoning for splitting IEventNotification and IEventContext into separate interfaces? I thinks its a good idea, and probably makes it a bit easier to expand later on.

danielcirket commented 4 years ago

Although currently they are essentially the same interface in terms of properties, it made more sense grammatically, and with how concepts are described elsewhere in articles and such to publish a notification than to publish a context if that makes sense. It does allow them to diverge in the future but that's also not super likely.