search

fabriziosestito / commanded-spear-adapter

Spear event store adapter for Commanded
MIT License
19 stars 5 forks source link

Make sure to read the last snapshot and fix deserializing data #91

Closed philipgiuliani closed 8 months ago

philipgiuliani commented 9 months ago

Issue 1: Even though the combination of start and backwards seems to make sense, this implementation was not correct. The EventStore returns the first snapshot in this case instead of the last snapshot. Unfortunately this doesn't seem to be covered by the Commanded Testsuite.

Issue 2: The snapshot.data was not decoded to a struct but just to a map, which caused the ProcessManager to not match any of the handle clauses.

philipgiuliani commented 8 months ago

No problem, thanks for taking a look! ☺️