Page MenuHomePhabricator
Differential D11190

[lib/native/web] read communites from sqlite on app start
ClosedPublic
Actions

Authored by ginsu on Feb 27 2024, 11:35 PM.
Tags
None
Referenced Files
F1800153: D11190.diff
Mon, May 20, 5:47 AM
Unknown Object (File)
Fri, May 17, 5:52 PM
Unknown Object (File)
Mon, May 13, 12:33 AM
Unknown Object (File)
Mon, May 13, 12:33 AM
Unknown Object (File)
Mon, May 13, 12:33 AM
Unknown Object (File)
Mon, May 13, 12:33 AM
Unknown Object (File)
Sun, May 12, 11:31 PM
Unknown Object (File)
Apr 6 2024, 11:21 PM
View All 23 Files
Subscribers
ashoat
tomek

Details

Reviewers
atul
inka
kamil
Commits
rCOMMcace3a54d8b3: [lib/native/web] read communites from sqlite on app start
Summary

This code gets the communty data from the client db and populates it into the redux store

Linear task: https://linear.app/comm/issue/ENG-6536/read-communities-from-sqlite-on-app-start

Depends on D11166

Test Plan

Confrimed that the community data in redux matches what is found in the client db

Diff Detail

Repository
rCOMM Comm
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

ginsu created this revision.Feb 27 2024, 11:35 PM
Herald added subscribers: tomek, ashoat. · View Herald TranscriptFeb 27 2024, 11:35 PM
ginsu edited the summary of this revision. (Show Details)Feb 27 2024, 11:37 PM
ginsu added reviewers: atul, inka, kamil.
ginsu added a parent revision: D11166: [lib] introduce community reducer.
Harbormaster completed remote builds in B27220: Diff 37679.Feb 27 2024, 11:51 PM
ginsu requested review of this revision.Feb 27 2024, 11:51 PM
ginsu added a child revision: D11191: DRAFT [lib/native/web] introduce migration to populate community store.Feb 28 2024, 12:19 AM
kamil accepted this revision.Feb 28 2024, 1:00 AM
This revision is now accepted and ready to land.Feb 28 2024, 1:00 AM
kamil added a comment.Feb 28 2024, 1:01 AM

Confrimed that the community data in redux matches what is found in the client db

was it tested on both the web and native?

ginsu added a comment.Mar 4 2024, 12:07 PM

was it tested on both the web and native?

Yes confirmed that I was able to read and see communities on both web and native from sqlite

ginsu updated this revision to Diff 37797.Mar 4 2024, 12:34 PM

rebase before landing

This revision was landed with ongoing or failed builds.Mar 4 2024, 12:39 PM
Closed by commit rCOMMcace3a54d8b3: [lib/native/web] read communites from sqlite on app start (authored by ginsu). · Explain Why
This revision was automatically updated to reflect the committed changes.
ginsu added a commit: rCOMMcace3a54d8b3: [lib/native/web] read communites from sqlite on app start.
Harbormaster completed remote builds in B27315: Diff 37797.Mar 4 2024, 1:35 PM

Revision Contents
Changeset List

PathSize
lib/
reducers/
20 lines
1 line
types/
2 lines
native/
data/
5 lines
web/
database/
utils/
9 lines

Diff 37797

View Options

lib/reducers/community-reducer.js

Loading...
View Options

lib/reducers/message-reducer.test.js

Loading...
View Options

lib/types/store-ops-types.js

Loading...
View Options

native/data/sqlite-data-handler.js

Loading...
View Options

web/database/utils/store.js

Loading...