Page MenuHomePhabricator

[lib] Remove community from thick threads
ClosedPublic

Authored by tomek on Aug 5 2024, 3:24 AM.
Tags
None
Referenced Files
Unknown Object (File)
Wed, Nov 6, 11:33 PM
Unknown Object (File)
Wed, Nov 6, 11:33 PM
Unknown Object (File)
Wed, Nov 6, 11:33 PM
Unknown Object (File)
Wed, Nov 6, 10:34 PM
Unknown Object (File)
Sat, Nov 2, 4:57 AM
Unknown Object (File)
Sat, Nov 2, 4:57 AM
Unknown Object (File)
Sat, Nov 2, 4:56 AM
Unknown Object (File)
Sat, Nov 2, 4:37 AM
Subscribers

Details

Summary
Test Plan

Made sure that it's possible to create a sidebar. Also checked if persisting and reading thin and thick threads from SQLite works on web.

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

lib/utils/thread-ops-utils.js
47 ↗(On Diff #43067)

This is for Flow. ClientDBThreadInfo has community as a mandatory nullable property, but thick threads don't have it at all.

native/redux/persist.js
405–427 ↗(On Diff #43067)

We shouldn't expect any thick thread to be in the store while performing this old migration.

Harbormaster returned this revision to the author for changes because remote builds failed.Aug 5 2024, 3:45 AM
Harbormaster failed remote builds in B30887: Diff 43067!
tomek requested review of this revision.Aug 5 2024, 4:17 AM
tomek edited the summary of this revision. (Show Details)
This revision is now accepted and ready to land.Aug 5 2024, 6:23 AM
This revision was automatically updated to reflect the committed changes.