[Cashtab] Remove unnecessary wrapper function
ClosedPublic
Actions

Authored by bytesofman on Jul 27 2023, 20:32.

Details

Reviewers

Fabien

Group Reviewers

Restricted Project

Commits

rABC952fda96ea25: [Cashtab] Remove unnecessary wrapper function

Summary

T3154

This was needed once upon a time but its functionality is now included in the utxo-lib library.

Test Plan

npm test
cd cashtab/ && grep -r encodeOpReturnScript src/ and no output

Diff Detail

Repository

rABC Bitcoin ABC

Branch

cashtab-deprecate-encode

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 24607
Build 48811: Build Diff	cashtab-tests
Build 48810: arc lint + arc unit

Event Timeline

bytesofman created this revision.Jul 27 2023, 20:32

Herald added a reviewer: Restricted Project. · View Herald TranscriptJul 27 2023, 20:32

bytesofman requested review of this revision.Jul 27 2023, 20:32

bytesofman edited the test plan for this revision. (Show Details)Jul 27 2023, 20:33

Harbormaster completed remote builds in B24607: Diff 41610.Jul 27 2023, 20:35

Fabien accepted this revision.Jul 28 2023, 08:41

This revision is now accepted and ready to land.Jul 28 2023, 08:41

Closed by commit rABC952fda96ea25: [Cashtab] Remove unnecessary wrapper function (authored by bytesofman). · Explain WhyJul 28 2023, 17:08

This revision was automatically updated to reflect the committed changes.

bytesofman added a commit: rABC952fda96ea25: [Cashtab] Remove unnecessary wrapper function.

Revision Contents
Changeset List

Path

Size

cashtab/

src/

utils/

cashMethods.js

11 lines

Diff 41610

View Options

cashtab/src/utils/cashMethods.js

[Cashtab] Remove unnecessary wrapper functionClosedPublicActions