[ecash-herald] Patch bug in daily summary calc
ClosedPublic
Actions

Authored by bytesofman on Oct 20 2024, 02:55.

Details

Reviewers

emack

Group Reviewers

Restricted Project

Commits

rABC6f457b103ee2: [ecash-herald] Patch bug in daily summary calc

Summary

Typo resulting in getting last 48 hrs worth of blocks instead of 24, patch

The checker was working, but bc I was mixing up "yesterday" with "today" -- was passing "yesterday" into getAllBlockTxs, which then had to go back in time well more than 144 blocks to get the "right" (wrong) block.

Test Plan

npm test, confirm typo corrected and the "right" date is going into getAllBlockTxs

Diff Detail

Repository

rABC Bitcoin ABC

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

bytesofman created this revision.Oct 20 2024, 02:55

Herald added a reviewer: Restricted Project. · View Herald TranscriptOct 20 2024, 02:55

back out debug condition change

bytesofman published this revision for review.Oct 20 2024, 02:56

Harbormaster completed remote builds in B30695: Diff 50229.Oct 20 2024, 02:57

Harbormaster completed remote builds in B30696: Diff 50230.

emack accepted this revision.Oct 20 2024, 06:06

This revision is now accepted and ready to land.Oct 20 2024, 06:06

Closed by commit rABC6f457b103ee2: [ecash-herald] Patch bug in daily summary calc (authored by bytesofman). · Explain WhyOct 20 2024, 10:10

This revision was automatically updated to reflect the committed changes.

bytesofman added a commit: rABC6f457b103ee2: [ecash-herald] Patch bug in daily summary calc.

Revision Contents
Changeset List

Path

Size

apps/

ecash-herald/

src/

events.js

4 lines

Diff 50231

View Options

apps/ecash-herald/src/events.js

[ecash-herald] Patch bug in daily summary calcClosedPublicActions