Page MenuHomePhabricator

[web] Don't render ThreadListProvider in CommunityDrawer
ClosedPublic

Authored by ashoat on Nov 22 2024, 7:09 AM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Dec 27, 10:09 PM
Unknown Object (File)
Fri, Dec 27, 3:49 PM
Unknown Object (File)
Fri, Dec 27, 3:49 PM
Unknown Object (File)
Fri, Dec 27, 3:48 PM
Unknown Object (File)
Fri, Dec 27, 3:43 PM
Unknown Object (File)
Fri, Dec 27, 11:03 AM
Unknown Object (File)
Fri, Dec 27, 10:16 AM
Unknown Object (File)
Fri, Dec 27, 9:21 AM
Subscribers
None

Details

Summary

Searched the codebase and couldn't find any use of ThreadListContext within this component tree. It was initially introduced in D6162, but I couldn't find any uses in that stack either.

This is an expensive component to render, so we should avoid rendering it unless necessary.

Test Plan
  1. I did a thorough search for all consumers of ThreadListContext and confirmed that there were none in this component tree.
  2. I tested the web app in my local environment and didn't see any issues.
  3. I tested the web app in a production setting (my personal server at comm.domains, pointed at my production keyserver on AWS) and didn't see any issues.

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

I also searched the codebase and didn't see any usage.

This revision is now accepted and ready to land.Nov 25 2024, 3:21 AM