Paths

Table of Contentst

Differential D9683

[native] stop persisting `messageStore.threads` via redux-persist
ClosedPublic
Actions

Authored by kamil on Nov 3 2023, 1:49 PM.

Details

Reviewers

atul

Commits

rCOMM84c5f15e4a68: [native] stop persisting `messageStore.threads` via redux-persist

Summary

ENG-3743

We have this data in SQLite now.

Depends on D9682

Test Plan

Try closing and opening the app multiple times to make sure there are no weird errors.

Diff Detail

Repository

rCOMM Comm

Branch

stop-persist-msg

Lint

No Lint Coverage

Unit

No Test Coverage

Event Timeline

kamil created this revision.Nov 3 2023, 1:49 PM

kamil held this revision as a draft.

Herald added subscribers: will, tomek, ashoat. · View Herald TranscriptNov 3 2023, 1:49 PM

kamil published this revision for review.Nov 3 2023, 2:00 PM

Harbormaster completed remote builds in B23715: Diff 32674.Nov 3 2023, 2:10 PM

atul accepted this revision.Nov 3 2023, 6:33 PM

atul added inline comments.

native/redux/persist.js
989	Is your editor auto formatter maybe adding these newlines? Feel free to disregard if intentional, but noticed these D9682 as well.

This revision is now accepted and ready to land.Nov 3 2023, 6:33 PM

kamil added inline comments.Nov 6 2023, 5:22 PM

native/redux/persist.js
989	looks like it's my IDE fault, it is adding this while committing, I'll take a look

remove blank lines

rebase

atul added inline comments.Nov 7 2023, 5:14 PM

native/redux/persist.js
989	Sweet, thanks for taking a look! Not a huge deal regardless, guess it clutters up `git blame` history a bit.. but not end of the world

This revision was landed with ongoing or failed builds.Nov 8 2023, 11:42 AM

Closed by commit rCOMM84c5f15e4a68: [native] stop persisting `messageStore.threads` via redux-persist. · Explain Why

This revision was automatically updated to reflect the committed changes.

kamil added a commit: rCOMM84c5f15e4a68: [native] stop persisting `messageStore.threads` via redux-persist.

Harbormaster completed remote builds in B23867: Diff 32895.Nov 8 2023, 2:50 PM

Harbormaster completed remote builds in B23869: Diff 32897.Nov 8 2023, 2:54 PM

Revision Contents
Changeset List

Path

Size

native/

redux/

persist.js

28 lines

Diff 32674

View Options

[native] stop persisting `messageStore.threads` via redux-persistClosedPublicActions