Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@valentinbeggi
Copy link
Contributor

@valentinbeggi valentinbeggi commented Aug 26, 2022

TODO before merging:

  • Check with demo that everything works as expected (add console.logs everywhere) !

Then, before next release:

  • Add recomputeSnapshot method on EventStore class.
  • Add reducer hash in EventStore props and store it in snapshots + validate on read that it is still up-to-date (otherwise, recompute snapshot)
  • Create recomputeSnapshots method on EventStore class that iterates on aggregateIds and snapshots, and recompute them if reducer hash is not up-to-date

@ThomasAribart ThomasAribart changed the title 🚧 ✨ add snapshot configuration in EventStore feature: add snapshot configuration in EventStore Sep 16, 2022
@github-actions github-actions bot added the minor label Sep 16, 2022
@github-actions github-actions bot added minor and removed minor labels Sep 23, 2022
@github-actions github-actions bot added minor and removed minor labels Sep 23, 2022
@github-actions github-actions bot added minor and removed minor labels Sep 23, 2022
@ThomasAribart ThomasAribart merged commit 1bf3217 into main Sep 23, 2022
@ThomasAribart ThomasAribart deleted the feat/add-snapshot branch September 23, 2022 17:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants