Page MenuHomePhabricator
Differential D12148

Connect pipes between C++ and JS for entry store
ClosedPublic
Actions

Authored by marcin on May 21 2024, 7:42 AM.
Tags
None
Referenced Files
F2132504: D12148.diff
Thu, Jun 27, 10:59 PM
Unknown Object (File)
Fri, Jun 21, 5:29 AM
Unknown Object (File)
Thu, Jun 20, 5:29 PM
Unknown Object (File)
Thu, Jun 20, 5:29 PM
Unknown Object (File)
Thu, Jun 20, 5:26 PM
Unknown Object (File)
Thu, Jun 20, 4:56 PM
Unknown Object (File)
Wed, Jun 19, 9:49 AM
Unknown Object (File)
Mon, Jun 17, 9:52 AM
View All 31 Files
Subscribers
ashoat

Details

Reviewers
tomek
kamil
Commits
rCOMMb51b09fb1359: Connect pipes between C++ and JS for entry store
Summary

This differential enables fetching data from SQLite to redux and reducers to create ops for entry store

Test Plan

flow

Diff Detail

Repository
rCOMM Comm
Branch
marcin-tomek/eng-3486
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

marcin created this revision.May 21 2024, 7:42 AM
Herald added a subscriber: ashoat. · View Herald TranscriptMay 21 2024, 7:42 AM
marcin added a parent revision: D12142: Implement JSI for EntryStore.May 21 2024, 7:43 AM
tomek added a child revision: D12143: [lib] Refactor the entries reducer to return an object.May 21 2024, 7:52 AM
Harbormaster returned this revision to the author for changes because remote builds failed.May 21 2024, 8:15 AM
Harbormaster failed remote builds in B29106: Diff 40472!
marcin updated this revision to Diff 40498.May 21 2024, 12:09 PM

Fix CI

Harbormaster completed remote builds in B29130: Diff 40498.May 21 2024, 12:26 PM
marcin requested review of this revision.May 21 2024, 12:26 PM
tomek accepted this revision.May 22 2024, 2:42 AM
tomek added inline comments.
web/shared-worker/worker/process-operations.js
554 ↗(On Diff #40498)
This revision is now accepted and ready to land.May 22 2024, 2:42 AM
marcin updated this revision to Diff 40570.May 22 2024, 8:41 AM

Change & -> $

Harbormaster completed remote builds in B29199: Diff 40570.May 22 2024, 9:51 AM
Closed by commit rCOMMb51b09fb1359: Connect pipes between C++ and JS for entry store (authored by marcin). · Explain WhyMay 27 2024, 5:45 AM
This revision was automatically updated to reflect the committed changes.
marcin added a commit: rCOMMb51b09fb1359: Connect pipes between C++ and JS for entry store.

Revision Contents
Changeset List

PathSize
lib/
reducers/
1 line
types/
10 lines
native/
data/
5 lines
redux/
5 lines
web/
shared-worker/
utils/
16 lines
worker/
41 lines

Diff 40472

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

native/redux/redux-utils.js

Loading...
View Options

web/shared-worker/utils/store.js

Loading...
View Options

web/shared-worker/worker/process-operations.js

Loading...