Page MenuHomePhabricator

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

Authored by atul on Jan 16 2024, 1:31 PM.
Tags
None
Referenced Files
F3390741: D10655.id36009.diff
Sat, Nov 30, 12:55 AM
Unknown Object (File)
Tue, Nov 26, 6:45 PM
Unknown Object (File)
Sun, Nov 24, 2:36 PM
Unknown Object (File)
Sun, Nov 17, 12:06 PM
Unknown Object (File)
Mon, Nov 11, 6:15 PM
Unknown Object (File)
Sat, Nov 9, 2:00 PM
Unknown Object (File)
Thu, Nov 7, 5:27 AM
Unknown Object (File)
Tue, Nov 5, 3:40 PM
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.