Page MenuHomePhabricator
Differential D7938

[native] fetch reports from SQLite and dispatch to store
ClosedPublic
Actions

Authored by kamil on Tue, May 23, 3:44 AM.
Tags
None
Referenced Files
Unknown Object (File)
Mon, May 29, 3:54 AM
Unknown Object (File)
Tue, May 23, 5:42 PM
Subscribers
ashoat
atul

Details

Reviewers
tomek
michal
Commits
rCOMM2cb7cef3cfe0: [native] fetch reports from SQLite and dispatch to store
Summary

Retrieving data and putting back to store.
We could consider moving this logic to different place - reports are not needed on app start.

Depends on D7937

Test Plan

Check if reports are properly fetched and converted.

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

kamil created this revision.Tue, May 23, 3:44 AM
kamil held this revision as a draft.
Herald added subscribers: atul, ashoat. · View Herald TranscriptTue, May 23, 3:44 AM
kamil added a child revision: D7939: [native] stop persisting reports.Tue, May 23, 4:00 AM
Harbormaster completed remote builds in B19650: Diff 26858.Tue, May 23, 4:03 AM
kamil published this revision for review.Tue, May 23, 4:55 PM
tomek accepted this revision.Wed, May 24, 3:19 AM

We could consider moving this logic to different place - reports are not needed on app start.

Yeah, waiting for these when the app starts isn't ideal.

This revision is now accepted and ready to land.Wed, May 24, 3:19 AM
kamil updated this revision to Diff 27298.Wed, May 31, 8:32 AM

rebase before landing

kamil added a comment.Wed, May 31, 8:33 AM
In D7938#235912, @tomek wrote:

We could consider moving this logic to different place - reports are not needed on app start.

Yeah, waiting for these when the app starts isn't ideal.

created task: ENG-4021

Harbormaster completed remote builds in B19923: Diff 27298.Wed, May 31, 8:51 AM
Closed by commit rCOMM2cb7cef3cfe0: [native] fetch reports from SQLite and dispatch to store (authored by kamil). · Explain WhyWed, May 31, 9:24 AM
This revision was automatically updated to reflect the committed changes.
kamil added a commit: rCOMM2cb7cef3cfe0: [native] fetch reports from SQLite and dispatch to store.

Revision Contents
Changeset List

PathSize
lib/
ops/
7 lines
reducers/
1 line
25 lines
types/
2 lines
native/
data/
7 lines

Diff 27306

View Options

lib/ops/report-store-ops.js

Loading...
View Options

lib/reducers/message-reducer.test.js

Loading...
View Options

lib/reducers/report-store-reducer.js

Loading...
View Options

lib/types/redux-types.js

Loading...
View Options

native/data/sqlite-data-handler.js

Loading...