HomePhabricator
Diffusion Comm 069f7d738f1d

[native] Replace `componentDidUpdate` with `useEffect`s in…

Description

[native] Replace componentDidUpdate with useEffects in ConnectedChatThreadList

Summary:
Replace componentDidUpdate with three useEffects that should fire in the same conditions.

This diff is just one step in the process of converting ChatThreadList into a functional component. When this work is done, we will be able to avoid a lot of re-rendering and hopefully improve performance quite a bit.


Depends on D9189

Test Plan:
ChatThreadList + search experience continue to work as expected.

Specifically, scrollToTop behavior was working as expected.

Reviewers: ginsu, tomek, rohan, ashoat

Reviewed By: ashoat

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