[Cashtab] notification duration set as app parameter
ClosedPublic
Actions

Authored by kieran709 on Oct 28 2021, 22:22.

Details

Reviewers

bytesofman

Group Reviewers

Restricted Project

Commits

rABC3ac99c4fd2a9: [Cashtab] notification duration set as app parameter

Summary

Added notification duration as parameter in Ticker.js. Related to task T1921

Test Plan

cd web/cashtab
npm start
send XEC, ensure notification duration is correct,
create token, ensure notification duration is correct,
send token, ensure notification duration is correct,
receive XEC, ensure notification duration is correct,
receive token, ensure notification duration is correct,

Diff Detail

Repository

rABC Bitcoin ABC

Branch

notification-duration-parameter

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 17134
Build 34099: Build Diff	cashtab-tests
Build 34098: arc lint + arc unit

Event Timeline

kieran709 created this revision.Oct 28 2021, 22:22

Herald added a reviewer: Restricted Project. · View Herald TranscriptOct 28 2021, 22:22

kieran709 requested review of this revision.Oct 28 2021, 22:22

Harbormaster completed remote builds in B17134: Diff 30691.Oct 28 2021, 22:25

kieran709 retitled this revision from notification duration set as app parameter to [Cashtab] notification duration set as app parameter.Oct 28 2021, 22:43

bytesofman requested changes to this revision.Oct 28 2021, 22:55

bytesofman added inline comments.