Page MenuHomePhabricator

Rename `useDisconnectedBarVisibilityHandler` to `useKeyserverReachabilityHandler`
ClosedPublic

Authored by atul on Jan 24 2024, 10:47 PM.
Tags
None
Referenced Files
Unknown Object (File)
Mar 9 2024, 11:33 AM
Unknown Object (File)
Mar 9 2024, 11:33 AM
Unknown Object (File)
Mar 5 2024, 2:57 PM
Unknown Object (File)
Mar 5 2024, 2:53 PM
Unknown Object (File)
Mar 5 2024, 2:53 PM
Unknown Object (File)
Mar 5 2024, 2:53 PM
Unknown Object (File)
Mar 5 2024, 2:53 PM
Unknown Object (File)
Feb 1 2024, 1:31 PM
Subscribers
None

Details

Summary

Part of https://linear.app/comm/issue/ENG-5796/update-usedisconnectedbarvisibilityhandler

Basically we want to maintain the functionality that determines whether a keyserver is reachable, but we want to rename things so they're no disconnected bar sepcific (which will now display or not display based on general network connectivity).

Will need to make some Redux changes as well.


Depends on D10811

Test Plan

NA, flow


Manually killed/started local keyserver a few times and ensured that Redux state updated as expected.

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

atul requested review of this revision.Jan 24 2024, 11:06 PM
tomek requested changes to this revision.Jan 25 2024, 3:00 AM

This diff should have a meaningful test plan

This revision now requires changes to proceed.Jan 25 2024, 3:00 AM
atul requested review of this revision.Jan 25 2024, 11:38 AM

Updated Test Plan

This revision is now accepted and ready to land.Jan 26 2024, 2:09 AM