Page MenuHomePhabricator

Expand `ResolvedThreadInfo` alias to `[Legacy/MinimallyEncoded]ResolvedThreadInfo`
ClosedPublic

Authored by atul on Jan 16 2024, 1:31 PM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, May 30, 3:36 AM
Unknown Object (File)
Thu, May 23, 2:23 PM
Unknown Object (File)
Mon, May 20, 6:02 AM
Unknown Object (File)
Mon, May 20, 6:02 AM
Unknown Object (File)
Mon, May 20, 6:02 AM
Unknown Object (File)
Mon, May 20, 6:02 AM
Unknown Object (File)
Mon, May 20, 6:02 AM
Unknown Object (File)
Mon, May 20, 5:57 AM
Subscribers
None

Details

Summary

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

Unlike [Raw]ThreadInfo, we will be able to completely remove LegacyResolvedThreadInfo since it's a "UI type" and isn't persisted anywhere or anything.

Did this via WebStorm "Inline alias" refactor tool.


Depends on D10654

Test Plan

flow

Used automated IDE tool for this, but did have to correct one thing:

5f440b.png (230×1 px, 63 KB)

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

atul requested review of this revision.Jan 16 2024, 1:50 PM
This revision is now accepted and ready to land.Jan 16 2024, 7:01 PM

There was some extraneous re-ordering of imports in this diff. Removed those changes.

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