[Cashtab] Remove unused chronik param from Ticker.js
ClosedPublic
Actions

Authored by bytesofman on May 15 2023, 21:44.

Details

Reviewers

emack

Group Reviewers

Restricted Project

Commits

rABC7e407f6998e2: [Cashtab] Remove unused chronik param from Ticker.js

Summary

Depends on D13901

Param was used in first approach to implementing aliases in Cashtab. No longer used.

Test Plan

npm test
grep -r chronikTxsPerPage src/ and no output

Diff Detail

Repository

rABC Bitcoin ABC

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

bytesofman created this revision.May 15 2023, 21:44

Herald added a reviewer: Restricted Project. · View Herald TranscriptMay 15 2023, 21:44

bytesofman requested review of this revision.May 15 2023, 21:44

Harbormaster completed remote builds in B23719: Diff 40319.May 15 2023, 21:47

bytesofman added a child revision: D13904: [Cashtab] Minimize tx fee.May 15 2023, 22:04

emack accepted this revision.May 16 2023, 06:09

This revision is now accepted and ready to land.May 16 2023, 06:09

Closed by commit rABC7e407f6998e2: [Cashtab] Remove unused chronik param from Ticker.js (authored by bytesofman). · Explain WhyMay 17 2023, 03:29

This revision was automatically updated to reflect the committed changes.

bytesofman added a commit: rABC7e407f6998e2: [Cashtab] Remove unused chronik param from Ticker.js.

Revision Contents
Changeset List

Path

Size

cashtab/

src/

components/

Common/

Ticker.js

1 line

Diff 40332

View Options

cashtab/src/components/Common/Ticker.js

[Cashtab] Remove unused chronik param from Ticker.jsClosedPublicActions