Use new JSI call in SQLiteDataHandler
ClosedPublic
Actions

Authored by marcin on Nov 30 2022, 11:32 AM.

Details

Reviewers

tomek
atul

Commits

rCOMM08033b5cdbdf: Use new JSI call in SQLiteDataHandler

Summary

This differential replaces separate calls for each SQLite data with new bulk JSI call.

Test Plan

Ensure in flipper SET_CLIENT_DB_STORE action payload is correct

Diff Detail

Repository

rCOMM Comm

Branch

marcin/eng-2137

Lint

No Lint Coverage

Unit

No Test Coverage

Event Timeline

marcin created this revision.Nov 30 2022, 11:32 AM

Herald added a subscriber: ashoat. · View Herald TranscriptNov 30 2022, 11:32 AM

marcin added a parent revision: D5774: Implement bulk JSI call te retrieve all SQLite data.Nov 30 2022, 11:34 AM

marcin added a child revision: D5776: Remove unecessary JSI queries.Nov 30 2022, 11:44 AM

Harbormaster completed remote builds in B13889: Diff 19029.Nov 30 2022, 11:44 AM

marcin requested review of this revision.Nov 30 2022, 11:44 AM

tomek accepted this revision.Dec 1 2022, 6:22 AM

This revision is now accepted and ready to land.Dec 1 2022, 6:22 AM

Rebase before landing

Harbormaster completed remote builds in B13938: Diff 19098.Dec 2 2022, 12:56 AM

Closed by commit rCOMM08033b5cdbdf: Use new JSI call in SQLiteDataHandler (authored by marcin). · Explain WhyDec 2 2022, 1:06 AM

This revision was automatically updated to reflect the committed changes.

marcin added a commit: rCOMM08033b5cdbdf: Use new JSI call in SQLiteDataHandler.

Revision Contents
Changeset List

Path

Size

native/

data/

sqlite-data-handler.js

10 lines

Diff 19029

View Options

native/data/sqlite-data-handler.js

Use new JSI call in SQLiteDataHandlerClosedPublicActions