search

danielcirket / OpenEventSourcing

MIT License
3 stars 0 forks source link

Usability: DefaultEventModelFactory expects causation id to be non-null #26

Closed danielcirket closed 4 years ago

danielcirket commented 4 years ago

There may be cases where you will not have a causation id when storing an event, such as events being raised without being directly caused by a command - think non command based functionality, or for things like external logins from a third party identity provider.

We should probably allow this to be null to match the definition on the IEvent interface as seen here

danielcirket commented 4 years ago

Correlation Id is technically nullable on IEvent too, so probably should include both properties for this.

/cc @AtLeastITry