[Cashtab] Only show extension install notification for webapp
ClosedPublic
Actions

Authored by bytesofman on Sat, Nov 22, 01:00.

Details

Reviewers

Fabien

Group Reviewers

Restricted Project

Commits

rABC6ba316c7938c: [Cashtab] Only show extension install notification for webapp

Summary

This was showing up in the android app, make sure it's web only

Test Plan

npm test, ./start-android.sh with a tablet connected and DO NOT see the ext msg, in webapp we do if it is not installed (check incognito)

Diff Detail

Repository

rABC Bitcoin ABC

Branch

app-no-ext-msg

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 35145
Build 69748: Build Diff	cashtab-tests
Build 69747: arc lint + arc unit

Event Timeline

bytesofman created this revision.Sat, Nov 22, 01:00

Herald added a reviewer: Restricted Project. · View Herald TranscriptSat, Nov 22, 01:00

bytesofman published this revision for review.Sat, Nov 22, 01:01

Harbormaster completed remote builds in B35145: Diff 56725.Sat, Nov 22, 01:05

Fabien accepted this revision.Sat, Nov 22, 02:33

This revision is now accepted and ready to land.Sat, Nov 22, 02:33

Closed by commit rABC6ba316c7938c: [Cashtab] Only show extension install notification for webapp (authored by bytesofman). · Explain WhySat, Nov 22, 04:39

This revision was automatically updated to reflect the committed changes.

bytesofman added a commit: rABC6ba316c7938c: [Cashtab] Only show extension install notification for webapp.

Revision Contents
Changeset List

Path

Size

cashtab/

src/

components/

AppModes/

WebApp.tsx

3 lines

Diff 56725

View Options

cashtab/src/components/AppModes/WebApp.tsx

[Cashtab] Only show extension install notification for webappClosedPublicActions