New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

"Opinionated" CommonDomain #2

Open

andreabalducci opened this issue May 29, 2015 · 1 comment

Labels

Member

andreabalducci commented May 29, 2015

In our projects we have forked common domain to

have typed ids instead of guids
have a clear separation from Aggregate and his state
support automatic snapshots
check invariants
simplify testing

Will be valuable for the project to have an "opitionated NEventStore.Domain" instead of the current one?

andreabalducci added enhancement question and removed enhancement labels

Member

AGiorgetti commented May 29, 2015

we've also did:

typed ids (with some optional mechanics to check coherence between aggregate type and id type, this is active only in debug builds)
tweaked snapshot support (we also have aggregate internal state in a separate class)
invariant checking (on demand or right before every save)
changed the aggregate construction so you must provide a valid Id for it to be created (an aggregate should not exist without an id)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment