Page MenuHomePhabricator

[lib] Remove support for `ServerThreadInfo` from `useKeyserverAdmin`
ClosedPublic

Authored by atul on Feb 6 2024, 12:21 PM.
Tags
None
Referenced Files
Unknown Object (File)
Apr 8 2024, 9:42 AM
Unknown Object (File)
Apr 8 2024, 9:42 AM
Unknown Object (File)
Apr 8 2024, 9:42 AM
Unknown Object (File)
Apr 8 2024, 9:42 AM
Unknown Object (File)
Feb 21 2024, 5:41 AM
Unknown Object (File)
Feb 21 2024, 3:37 AM
Unknown Object (File)
Feb 20 2024, 3:56 PM
Unknown Object (File)
Feb 16 2024, 6:58 PM
Subscribers
None

Details

Summary

In D10107 I encountered a scenario where flow wasn't able to type check ServerLegacyRoleInfo as part of a ServerThreadInfo being constructed.

To be safe, I decided to look at all instances of ServerThreadInfo and fetchServerThreadInfos() in the codebase to double check for similar issues.

I didn't find any, but noticed that useKeyserverAdmin accepted ServerThreadInfo which seems strange. Looks like it was added when useKeyserverAdmin was getKeyserverAdmin. Regardless, it's no longer necessary so I removed it.


Depends on D10107

Test Plan

flow + seeing that the hook is only ever used by the CommunityPill react component which should never encounter a ServerThreadInfo.

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable