[web] implement screen for testing Tunnelbroker
ClosedPublic
Actions

Authored by kamil on Oct 27 2023, 4:41 AM.

Details

Reviewers

michal
bartek
tomek
inka

Commits

rCOMM297e5ed63235: [web] implement screen for testing Tunnelbroker

Summary

This is a screen for testing Tunnelbroker utilities. It might be helpful for testing on production.

Styles/components are mostly copied from other places in the codebase which visually look similar. This is only for devs/staff and will be removed in the future.

Depends on D9616

Test Plan

See example here.

Diff Detail

Repository

rCOMM Comm

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

kamil created this revision.Oct 27 2023, 4:41 AM

kamil held this revision as a draft.

Herald added subscribers: will, ashoat. · View Herald TranscriptOct 27 2023, 4:41 AM

kamil added a reviewer: inka.Oct 27 2023, 4:42 AM

kamil added a child revision: D9618: [native] implement screen for testing Tunnelbroker.Oct 27 2023, 4:43 AM

Harbormaster completed remote builds in B23531: Diff 32456.Oct 27 2023, 5:16 AM

kamil published this revision for review.Oct 27 2023, 5:42 AM

michal added inline comments.Nov 3 2023, 4:19 AM

web/settings/tunnelbroker-message-list.react.js
20–24 ↗	(On Diff #32456)	We should cleanup previous listeners
29–39 ↗	(On Diff #32456)	That's a complicated ternary operator, can we extract it to a normal if?
web/settings/tunnelbroker-test.react.js
83 ↗	(On Diff #32456)	This probably makes sense