[lib] Use sqlite for message store local instead of redux persist
Needs RevisionPublic
Actions

Authored by will on Wed, Jun 12, 2:03 PM.

Details

Reviewers

tomek
kamil

Summary

This diff is in anticipation for when we want to use sqlite for message store local instead of redux persist. This should only be landed after the message store local sqlite diffs have been landed on master for a while and no errors are confirmed with the assertion function.

Depends on D12384

Test Plan

When landing, will verify that the sqlite data is properly reflected on app start on both native and web

Diff Detail

Repository

rCOMM Comm

Branch

sqlite/local_message_store

Lint

No Lint Coverage

Unit

No Test Coverage

Event Timeline

will created this revision.Wed, Jun 12, 2:03 PM

Herald added a subscriber: ashoat. · View Herald TranscriptWed, Jun 12, 2:03 PM

Harbormaster completed remote builds in B29633: Diff 41254.Wed, Jun 12, 2:20 PM

will requested review of this revision.Wed, Jun 12, 2:20 PM

It might be a good idea to create a task tracking landing this.

lib/reducers/message-reducer.js
1781	Are you really sure that `messageStore` will contain local messages from the payload? It looks like we're using them only in `!payloadMessages` case.