Details
Details
- Reviewers
tomek michal - Commits
- rCOMM9d47f0a1a686: [web] wrap store operations in transaction
- Run tests
- Make sure persistence still works
Diff Detail
Diff Detail
- Repository
- rCOMM Comm
- Branch
- add-transaction-on-web
- Lint
No Lint Coverage - Unit
No Test Coverage
Event Timeline
web/database/worker/process-operations.js | ||
---|---|---|
85–114 | I think this should all be in one transaction. See comment here:
|
Comment Actions
wrap everything in one transaction
web/database/worker/process-operations.js | ||
---|---|---|
85–114 | I wanted to match native but this also seems reasonable |
Comment Actions
I think we should probably update native to match this too. Ultimately we're trying to match Redux behavior... because our master-reducer code has eg. MessageStore depending on ThreadStore, it's important that changes between the two are in sync