[lib] make `useWaitForConnection` parametrized
ClosedPublic
Actions

Authored by kamil on Feb 18 2025, 5:01 PM.

Details

Reviewers

tomek
angelika

Commits

rCOMMa38076704885: [lib] make `useWaitForConnection` parametrized

Summary

Part of ENG-10186

We need to wait for only the Tunnelbroker connection to solve the issue described in the task linked above.

Test Plan

Tested in D14387

Diff Detail

Repository

rCOMM Comm

Branch

fixes

Lint

No Lint Coverage

Unit

No Test Coverage

Event Timeline

kamil created this revision.Feb 18 2025, 5:01 PM

kamil held this revision as a draft.

Herald added a subscriber: ashoat. · View Herald TranscriptFeb 18 2025, 5:01 PM

kamil added a child revision: D14386: [lib] fix `useWaitForConnection`.Feb 18 2025, 5:10 PM

kamil published this revision for review.Feb 18 2025, 5:13 PM

kamil edited the summary of this revision. (Show Details)

kamil edited the test plan for this revision. (Show Details)

Harbormaster completed remote builds in B33546: Diff 47152.Feb 18 2025, 5:24 PM

angelika accepted this revision.Feb 19 2025, 11:16 AM

This revision is now accepted and ready to land.Feb 19 2025, 11:16 AM

Closed by commit rCOMMa38076704885: [lib] make `useWaitForConnection` parametrized. · Explain WhyFeb 19 2025, 1:21 PM

This revision was automatically updated to reflect the committed changes.

kamil added a commit: rCOMMa38076704885: [lib] make `useWaitForConnection` parametrized.

Revision Contents
Changeset List

Path

Size

lib/

hooks/

relationship-hooks.js

2 lines

wait-for-connection.js

15 lines

Diff 47152

View Options

lib/hooks/relationship-hooks.js