Page MenuHomePhabricator

Rename `useDisconnectedBarVisibilityHandler` to `useKeyserverReachabilityHandler`
ClosedPublic

Authored by atul on Jan 24 2024, 10:47 PM.
Tags
None
Referenced Files
F3528140: D10812.id36341.diff
Tue, Dec 24, 8:19 AM
F3528139: D10812.id36330.diff
Tue, Dec 24, 8:19 AM
F3528138: D10812.id36125.diff
Tue, Dec 24, 8:19 AM
F3528120: D10812.id.diff
Tue, Dec 24, 8:19 AM
F3528110: D10812.diff
Tue, Dec 24, 8:19 AM
Unknown Object (File)
Tue, Dec 17, 8:10 PM
Unknown Object (File)
Tue, Nov 26, 7:36 PM
Unknown Object (File)
Nov 23 2024, 4:22 AM
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
No Lint Coverage
Unit
No Test Coverage

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