[Cashtab] Remove artifact CashtabSettings type from formatting
ClosedPublic
Actions

Authored by bytesofman on Wed, Jan 1, 12:36.

Details

Reviewers

Fabien

Group Reviewers

Restricted Project

Commits

rABCd5e2de595f89: [Cashtab] Remove artifact CashtabSettings type from formatting

Summary

This was defined in the formatting file before CashtabSettings was converted into a class.

It's no longer used here or elsewhere. Type is now defined in src/config/CashtabSettings.ts

Test Plan

npm test

Diff Detail

Repository

rABC Bitcoin ABC

Branch

remove-artifact-type

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 31863
Build 63218: Build Diff	cashtab-tests
Build 63217: arc lint + arc unit

Event Timeline

bytesofman created this revision.Wed, Jan 1, 12:36

Herald added a reviewer: Restricted Project. · View Herald TranscriptWed, Jan 1, 12:36

Harbormaster completed remote builds in B31863: Diff 51888.Wed, Jan 1, 12:42

bytesofman published this revision for review.Wed, Jan 1, 13:05

Fabien accepted this revision.Wed, Jan 1, 14:07

This revision is now accepted and ready to land.Wed, Jan 1, 14:07

Closed by commit rABCd5e2de595f89: [Cashtab] Remove artifact CashtabSettings type from formatting (authored by bytesofman). · Explain WhyWed, Jan 1, 15:18

This revision was automatically updated to reflect the committed changes.

bytesofman added a commit: rABCd5e2de595f89: [Cashtab] Remove artifact CashtabSettings type from formatting.

Revision Contents
Changeset List

Path

Size

cashtab/

src/

formatting/

index.ts

15 lines

Diff 51888

View Options

cashtab/src/formatting/index.ts

[Cashtab] Remove artifact CashtabSettings type from formattingClosedPublicActions