Page MenuHomePhabricator
Differential D12384

[lib] assert function and setting client db with sqlite data
ClosedPublic
Actions

Authored by will on Mon, Jun 10, 11:29 AM.
Tags
None
Referenced Files
F2109905: D12384.id41252.diff
Tue, Jun 25, 5:59 PM
Unknown Object (File)
Tue, Jun 25, 12:58 AM
Unknown Object (File)
Mon, Jun 24, 1:38 PM
Unknown Object (File)
Sun, Jun 23, 8:31 PM
Unknown Object (File)
Fri, Jun 21, 7:19 PM
Unknown Object (File)
Fri, Jun 21, 8:40 AM
Unknown Object (File)
Fri, Jun 21, 6:41 AM
Unknown Object (File)
Fri, Jun 21, 12:15 AM
View All 36 Files
Subscribers
ashoat
ginsu

Details

Reviewers
tomek
kamil
ginsu
Commits
rCOMM23c6aa4e6c09: [lib] assert function and setting client db with sqlite data
Summary

Populates the client db with message store local sqlite data and includes an assert function to verify the result of message store local operations

Depends on D12370

Test Plan

Tested on both native and web by

  1. launching app on prior migration version
  2. Failing to send several messages by disconnecting the keyserver
  3. Restarting app to confirm assertion between redux persist and sqlite data failure
  4. Running migration
  5. Verifying assertion now passes

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

will created this revision.Mon, Jun 10, 11:29 AM
Herald added a subscriber: ashoat. · View Herald TranscriptMon, Jun 10, 11:29 AM
will updated this revision to Diff 41173.Mon, Jun 10, 11:32 AM

remove whitespace

Harbormaster completed remote builds in B29581: Diff 41172.Mon, Jun 10, 11:47 AM
Harbormaster completed remote builds in B29582: Diff 41173.
will requested review of this revision.Mon, Jun 10, 11:49 AM
will edited the summary of this revision. (Show Details)Mon, Jun 10, 11:53 AM
tomek accepted this revision.Tue, Jun 11, 2:36 AM
tomek added inline comments.
lib/reducers/message-reducer.test.js
301 ↗(On Diff #41173)

What is the reason behind this change?

This revision is now accepted and ready to land.Tue, Jun 11, 2:36 AM
will planned changes to this revision.Tue, Jun 11, 2:27 PM

Realized this diff requires a critical change of using the action payload sqlite data

will added a reviewer: ginsu.Tue, Jun 11, 2:27 PM
will added a subscriber: ginsu.
will added inline comments.Tue, Jun 11, 2:32 PM
lib/reducers/message-reducer.test.js
301 ↗(On Diff #41173)

I did this to reflect how we handle messageStoreThreads where messageStoreThreads is actually converted into a map in the message reducer instead of in native/data/sqlite-data-handler.js

will updated this revision to Diff 41227.Tue, Jun 11, 3:53 PM

This diff actually uses sqlite data

This revision is now accepted and ready to land.Tue, Jun 11, 3:53 PM
will edited the test plan for this revision. (Show Details)Tue, Jun 11, 3:54 PM
will requested review of this revision.Tue, Jun 11, 4:08 PM
Harbormaster completed remote builds in B29615: Diff 41227.Tue, Jun 11, 4:11 PM
ginsu mentioned this in D12358: [lib] update message reducer to use ops approach for local field.Tue, Jun 11, 10:27 PM
tomek accepted this revision.Wed, Jun 12, 4:17 AM
tomek added inline comments.
lib/reducers/message-reducer.js
1630 ↗(On Diff #41227)

This transformation should probably be performed before dispatching setClientDBStoreActionType as we do for other data.

This revision is now accepted and ready to land.Wed, Jun 12, 4:17 AM
will updated this revision to Diff 41249.Wed, Jun 12, 12:17 PM

review feedback

Harbormaster failed remote builds in B29628: Diff 41249!Wed, Jun 12, 12:22 PM
will updated this revision to Diff 41252.Wed, Jun 12, 12:36 PM

rebase

will edited the test plan for this revision. (Show Details)Wed, Jun 12, 12:39 PM
Harbormaster completed remote builds in B29631: Diff 41252.Wed, Jun 12, 12:52 PM
will updated this revision to Diff 41253.Wed, Jun 12, 1:40 PM

set web app start to use sqlite

Harbormaster completed remote builds in B29632: Diff 41253.Wed, Jun 12, 1:57 PM
will added a child revision: D12409: [lib] Use sqlite for message store local instead of redux persist.Wed, Jun 12, 2:03 PM
will edited the test plan for this revision. (Show Details)Wed, Jun 12, 2:18 PM
Closed by commit rCOMM23c6aa4e6c09: [lib] assert function and setting client db with sqlite data (authored by will). · Explain WhyWed, Jun 12, 2:20 PM
This revision was automatically updated to reflect the committed changes.
will added a commit: rCOMM23c6aa4e6c09: [lib] assert function and setting client db with sqlite data.

Revision Contents
Changeset List

PathSize
lib/
reducers/
7 lines
41 lines
utils/
16 lines
native/
data/
6 lines
web/
shared-worker/
utils/
13 lines

Diff 41256

View Options

lib/reducers/master-reducer.js

Loading...
View Options

lib/reducers/message-reducer.js

Loading...
View Options

lib/utils/message-ops-utils.js

Loading...
View Options

native/data/sqlite-data-handler.js

Loading...
View Options

web/shared-worker/utils/store.js

Loading...