transmute-industries / transmute-framework

TypeScript dApp Framework
https://framework.transmute.industries/
42 stars 8 forks source link

Last Event Null / 0 Fail Maybe Update ReadModel #68

Closed OR13 closed 7 years ago

OR13 commented 7 years ago

There is a bug in maybe update readmodel, which reads the same event twice.

This causes problems when state transitions are patch algebra.

The bug can be hacked around by doing index checking in the reducer, but that should never be necessary.

although its not a terrible idea to enforce only new events will be processed by the reducer.

Need more tests for this sort of thing.