HomePhabricator
Diffusion Comm 7c75580841a3

[flow] Update `useResolvedThreadInfo` and fix subsequent `flow` issues

Description

[flow] Update useResolvedThreadInfo and fix subsequent flow issues

Summary:
As mentioned in D9896, there were quite a few places where refactoring code to accept MinimallyEncodedThreadInfo required the useResolvedThreadInfo hook to be updated. Updating that hook required a bunch of changes, which in turn required even more changes, etc.

There wasn't a super obvious way to break down the refactoring going into this since I was responding to flow issues as they surfaced, so apologize for the large diff. I think that the changes are straightforward enough that this is fine, but I can go back and chunk changes into separate diffs if that would make things easier to review.


Depends on D9896

Test Plan: Trusting flow, but also based on feedback here: https://phab.comm.dev/D9896#inline-61685... will make sure to read through this carefully a couple times and ensure the "base type(s)" and "minimally encoded type(s)" match up.

Reviewers: ashoat, ginsu, rohan, tomek

Reviewed By: ashoat

Differential Revision: https://phab.comm.dev/D9907

Details