Page MenuHomePhabricator

[lib] Remove most remaining instances of `LegacyThreadInfo` from `lib`
ClosedPublic

Authored by atul on Jan 17 2024, 4:26 PM.
Tags
None
Referenced Files
F1799873: D10677.id36066.diff
Mon, May 20, 4:51 AM
F1799872: D10677.id36061.diff
Mon, May 20, 4:51 AM
F1799871: D10677.id36055.diff
Mon, May 20, 4:51 AM
F1799870: D10677.id35746.diff
Mon, May 20, 4:51 AM
F1799865: D10677.id.diff
Mon, May 20, 4:51 AM
F1799864: D10677.diff
Mon, May 20, 4:51 AM
Unknown Object (File)
Mar 15 2024, 10:54 PM
Unknown Object (File)
Mar 7 2024, 5:28 AM
Subscribers
None

Details

Summary

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

NOTE: There are some remaining instances in lib related to message specs and in thread-utils that'll be handled in subsequent diffs for (maybe) easier review.

Depends on D10676

Test Plan

flow

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

atul requested review of this revision.Jan 17 2024, 4:45 PM
tomek added inline comments.
lib/selectors/nav-selectors.js
15–17 ↗(On Diff #35746)

Why did the ordering change?

This revision is now accepted and ready to land.Jan 18 2024, 3:05 AM

remove extraneous import changes

This revision was landed with ongoing or failed builds.Jan 23 2024, 1:43 PM
This revision was automatically updated to reflect the committed changes.