Page MenuHomePhabricator

[lib] Rename `RawThreadInfos` to `MixedRawThreadInfos`
ClosedPublic

Authored by atul on Jan 15 2024, 10:50 AM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, May 14, 10:00 PM
Unknown Object (File)
Sat, Apr 27, 6:13 PM
Unknown Object (File)
Thu, Apr 25, 10:10 AM
Unknown Object (File)
Mar 16 2024, 2:41 PM
Unknown Object (File)
Mar 5 2024, 5:19 PM
Unknown Object (File)
Mar 5 2024, 5:19 PM
Unknown Object (File)
Mar 5 2024, 5:19 PM
Unknown Object (File)
Mar 5 2024, 5:12 PM
Subscribers
None

Details

Summary

Part of https://linear.app/comm/issue/ENG-6366/rename-minimallyencodedrawthreadinfo-to-rawthreadinfo

New type looks like:

export type MixedRawThreadInfos = {
  +[id: string]: LegacyRawThreadInfo | RawThreadInfo,
};

which is slightly different than

LegacyRawThreadInfos | MinimallyEncodedRawThreadInfos because it allows values of both types.

However, we should be able to now replace most instances of MixedRawThreadInfos with LegacyRawThreadInfos | MinimallyEncodedRawThreadInfos. I think we only really will need MixedRawThreadInfos when we're working with SQLite store that may contain either (eg in case we're upgrading from an old version and applying migrations before encoding change).


Depends on D10634

Test Plan

flow

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable