[web] Move KeyserverConnectionHandler into ConnectedApp
ClosedPublic
Actions

Authored by bartek on Jan 15 2025, 7:32 AM.

Details

Reviewers

kamil
tomek

Commits

rCOMM85fe02279ea6: [web] Move KeyserverConnectionHandler into ConnectedApp

Summary

Fixes ENG-10087.
Moved the component deeper in the hierarchy, to be inside TunnelbrokerProvider

Test Plan

Web runs and no longer crashes on invariant.

Diff Detail

Repository

rCOMM Comm

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

bartek created this revision.Jan 15 2025, 7:32 AM

bartek held this revision as a draft.

Herald added a subscriber: ashoat. · View Herald TranscriptJan 15 2025, 7:32 AM

Harbormaster completed remote builds in B33245: Diff 46687.Jan 15 2025, 7:51 AM

bartek published this revision for review.Jan 15 2025, 7:57 AM

kamil accepted this revision.Jan 15 2025, 9:49 AM

This revision is now accepted and ready to land.Jan 15 2025, 9:49 AM

Closed by commit rCOMM85fe02279ea6: [web] Move KeyserverConnectionHandler into ConnectedApp. · Explain WhyJan 15 2025, 11:18 AM

This revision was automatically updated to reflect the committed changes.

bartek added a commit: rCOMM85fe02279ea6: [web] Move KeyserverConnectionHandler into ConnectedApp.

Revision Contents
Changeset List

Path

Size

web/

app.react.js

3 lines

root.js

3 lines

Diff 46689

View Options

web/app.react.js