Paths

Table of Contentst

Differential D12384

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

Authored by will on Jun 10 2024, 11:29 AM.

Tags

None

Referenced Files

	F3487137: D12384.id41256.diff
	Wed, Dec 18, 7:01 AM

	F3487136: D12384.id41253.diff
	Wed, Dec 18, 7:01 AM

	F3487135: D12384.id41252.diff
	Wed, Dec 18, 7:01 AM

	F3487134: D12384.id41249.diff
	Wed, Dec 18, 7:01 AM

	F3487133: D12384.id41172.diff
	Wed, Dec 18, 7:01 AM

	F3487132: D12384.id41227.diff
	Wed, Dec 18, 7:01 AM

	F3487131: D12384.id41173.diff
	Wed, Dec 18, 7:01 AM

	F3487121: D12384.id.diff
	Wed, Dec 18, 7:01 AM

Subscribers

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

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

Diff Detail

Repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

will created this revision.Jun 10 2024, 11:29 AM

Herald added a subscriber: ashoat. · View Herald TranscriptJun 10 2024, 11:29 AM

remove whitespace

Harbormaster completed remote builds in B29581: Diff 41172.Jun 10 2024, 11:47 AM

Harbormaster completed remote builds in B29582: Diff 41173.

will requested review of this revision.Jun 10 2024, 11:49 AM

will edited the summary of this revision. (Show Details)Jun 10 2024, 11:53 AM

tomek accepted this revision.Jun 11 2024, 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.Jun 11 2024, 2:36 AM

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

will added a reviewer: ginsu.Jun 11 2024, 2:27 PM

will added a subscriber: ginsu.

will added inline comments.Jun 11 2024, 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

This diff actually uses sqlite data

This revision is now accepted and ready to land.Jun 11 2024, 3:53 PM

will edited the test plan for this revision. (Show Details)Jun 11 2024, 3:54 PM

will requested review of this revision.Jun 11 2024, 4:08 PM

Harbormaster completed remote builds in B29615: Diff 41227.Jun 11 2024, 4:11 PM

ginsu mentioned this in D12358: [lib] update message reducer to use ops approach for local field.Jun 11 2024, 10:27 PM

tomek accepted this revision.Jun 12 2024, 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.Jun 12 2024, 4:17 AM

review feedback

Harbormaster failed remote builds in B29628: Diff 41249!Jun 12 2024, 12:22 PM

rebase

will edited the test plan for this revision. (Show Details)Jun 12 2024, 12:39 PM

Harbormaster completed remote builds in B29631: Diff 41252.Jun 12 2024, 12:52 PM

set web app start to use sqlite

Harbormaster completed remote builds in B29632: Diff 41253.Jun 12 2024, 1:57 PM

will added a child revision: D12409: [lib] Use sqlite for message store local instead of redux persist.Jun 12 2024, 2:03 PM

will edited the test plan for this revision. (Show Details)Jun 12 2024, 2:18 PM

Closed by commit rCOMM23c6aa4e6c09: [lib] assert function and setting client db with sqlite data (authored by will). · Explain WhyJun 12 2024, 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

Path

Size

lib/

reducers/

master-reducer.js

7 lines

message-reducer.js

41 lines

utils/

message-ops-utils.js

16 lines

native/

data/

sqlite-data-handler.js

6 lines

web/

shared-worker/

utils/

13 lines

Diff 41256

lib/reducers/master-reducer.js

Loading...

lib/reducers/message-reducer.js

Loading...

lib/utils/message-ops-utils.js

Loading...

native/data/sqlite-data-handler.js

Loading...

web/shared-worker/utils/store.js

Loading...