Paths

Table of Contentst

Differential D9683

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

Authored by kamil on Nov 3 2023, 6:49 AM.

Tags

None

Referenced Files

	F2190912: D9683.id.diff
	Thu, Jul 4, 1:03 PM

	Unknown Object (File)
	Sat, Jun 29, 2:25 PM

	Unknown Object (File)
	Sat, Jun 29, 3:19 AM

	Unknown Object (File)
	Fri, Jun 28, 11:51 PM

	Unknown Object (File)
	Sun, Jun 23, 11:15 AM

	Unknown Object (File)
	Sun, Jun 23, 6:32 AM

	Unknown Object (File)
	Sat, Jun 15, 4:18 PM

	Unknown Object (File)
	Fri, Jun 14, 6:51 AM

View All 37 Files

Subscribers

Details

Reviewers

Commits

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

Summary

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

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

kamil created this revision.Nov 3 2023, 6:49 AM

kamil held this revision as a draft.

Herald added subscribers: will, tomek, ashoat. · View Herald TranscriptNov 3 2023, 6:49 AM

kamil published this revision for review.Nov 3 2023, 7:00 AM

Harbormaster completed remote builds in B23715: Diff 32674.Nov 3 2023, 7:10 AM

atul accepted this revision.Nov 3 2023, 11:33 AM

atul added inline comments.

native/redux/persist.js
989 ↗	(On Diff #32674)	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, 11:33 AM

kamil added inline comments.Nov 6 2023, 9:22 AM

native/redux/persist.js
989 ↗	(On Diff #32674)	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, 9:14 AM

native/redux/persist.js
989 ↗	(On Diff #32674)	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, 3:42 AM

Closed by commit rCOMM84c5f15e4a68: [native] stop persisting `messageStore.threads` via redux-persist (authored by kamil). · 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, 6:50 AM

Harbormaster completed remote builds in B23869: Diff 32897.Nov 8 2023, 6:54 AM

Revision Contents
Changeset List

Path

Size

native/

redux/

26 lines

Diff 32934

native/redux/persist.js

Loading...