Page MenuHomePhabricator

[native] Fix useIsCurrentUserStaff hook
ClosedPublic

Authored by tomek on Mar 7 2023, 2:07 AM.
Tags
None
Referenced Files
Unknown Object (File)
Wed, Oct 30, 4:57 AM
Unknown Object (File)
Wed, Oct 30, 4:57 AM
Unknown Object (File)
Wed, Oct 30, 4:56 AM
Unknown Object (File)
Wed, Oct 30, 4:54 AM
Unknown Object (File)
Wed, Oct 30, 4:26 AM
Unknown Object (File)
Sat, Oct 19, 10:24 AM
Unknown Object (File)
Fri, Oct 18, 10:10 AM
Unknown Object (File)
Fri, Oct 18, 10:10 AM
Subscribers

Details

Summary

This hook was introduced in D5498 and incorrectly used useSelector from react-redux which isn't typed. That meant that the result of this hook was incorrectly typed as boolean while in fact it was something like undefined | null | string | boolean. The issue wasn't detected because we were simply checking truthiness of the value. In D6949 we started using this value as a query param and converted it to it using .toString method, which was failing when the returned value was null / undefined.

Test Plan

After updating the import Flow checks the types correctly. Also logged out and checked if fetching feature flags config works correctly.

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable