[landing] Remove `isDev` gate for "Launch app" icon in landing `Header`
ClosedPublic
Actions

Authored by atul on Apr 7 2022, 11:00 AM.

Details

Reviewers

ashoat
tomek

Commits

rCOMM307df3a5a399: [landing] Remove `isDev` gate for "Launch app" icon in landing `Header`

Summary

We were gating the icon behind an isDev flag until https://web.comm.app was ready to go, and now it is.

Test Plan

Clicked the button and it took me to https://web.comm.app

Diff Detail

Repository

rCOMM Comm

Branch

april7clean (branched from master)

Lint

No Lint Coverage

Unit

No Test Coverage

Event Timeline

atul created this revision.Apr 7 2022, 11:00 AM

Herald added subscribers: • abosh, • benschac, • karol and 2 others. · View Herald TranscriptApr 7 2022, 11:00 AM

Harbormaster completed remote builds in B7986: Diff 11191.Apr 7 2022, 11:04 AM

atul requested review of this revision.Apr 7 2022, 11:04 AM

atul added a reviewer: tomek.Apr 8 2022, 6:34 AM

ashoat accepted this revision.Apr 8 2022, 8:08 AM

This revision is now accepted and ready to land.Apr 8 2022, 8:08 AM

rebase before landing

This revision was landed with ongoing or failed builds.Apr 8 2022, 8:20 AM

Closed by commit rCOMM307df3a5a399: [landing] Remove `isDev` gate for "Launch app" icon in landing `Header` (authored by atul). · Explain Why

This revision was automatically updated to reflect the committed changes.

atul added a commit: rCOMM307df3a5a399: [landing] Remove `isDev` gate for "Launch app" icon in landing `Header`.

Harbormaster failed remote builds in B8032: Diff 11245!Apr 8 2022, 8:24 AM

Revision Contents
Changeset List

Path

Size

landing/

header.react.js

18 lines

Diff 11191

View Options

[landing] Remove `isDev` gate for "Launch app" icon in landing `Header`ClosedPublicActions