[ecash-herald] Remove useless info from prod logs
ClosedPublic
Actions

Authored by bytesofman on Jun 30 2023, 19:54.

Details

Reviewers

Fabien

Group Reviewers

Restricted Project

Commits

rABC9aa2c2ed7d29: [ecash-herald] Remove useless info from prod logs

Summary

T2972

These logs were used in debug testing but are not relevant in the prod app

Test Plan

Confirm only comments are removed, npm test still good

Diff Detail

Repository

rABC Bitcoin ABC

Branch

herald-cleanup-logs

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 24274
Build 48158: Build Diff	ecash-herald-tests
Build 48157: arc lint + arc unit

Event Timeline

bytesofman created this revision.Jun 30 2023, 19:54

Herald added a reviewer: Restricted Project. · View Herald TranscriptJun 30 2023, 19:54

bytesofman requested review of this revision.Jun 30 2023, 19:54

Harbormaster completed remote builds in B24274: Diff 41137.Jun 30 2023, 19:55

bytesofman added a child revision: D14176: [ecash-herald] Log success msgs by blockheight.Jun 30 2023, 20:13

Fabien accepted this revision.Jul 1 2023, 07:00

This revision is now accepted and ready to land.Jul 1 2023, 07:00

Closed by commit rABC9aa2c2ed7d29: [ecash-herald] Remove useless info from prod logs (authored by bytesofman). · Explain WhyJul 1 2023, 21:02

This revision was automatically updated to reflect the committed changes.

bytesofman added a commit: rABC9aa2c2ed7d29: [ecash-herald] Remove useless info from prod logs.

Revision Contents
Changeset List

Path

Size

apps/

ecash-herald/

src/

chronikWsHandler.js

3 lines

Diff 41137

View Options

apps/ecash-herald/src/chronikWsHandler.js

[ecash-herald] Remove useless info from prod logsClosedPublicActions