Differential D18077

Revert "[chronik] Electrum: add the blockchain.headers.subscribe endpoint"
ClosedPublic
Actions

Authored by Fabien on Mon, May 12, 13:11.

Details

Reviewers

PiRK

Group Reviewers

Restricted Project

Commits

rABCcdb17840db55: Revert "[chronik] Electrum: add the blockchain.headers.subscribe endpoint"

Summary

This reverts commit rABC9b02451660e2d4aa2b1394a4ece44e58981bb494.

There is a subtle difference in the notification format vs Fulcrum that I missed, this needs to be fixed.

Test Plan

ninja all check-all

Diff Detail

Repository

rABC Bitcoin ABC

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

Fabien created this revision.Mon, May 12, 13:11

Herald added a reviewer: Restricted Project. · View Herald TranscriptMon, May 12, 13:11

Fabien requested review of this revision.Mon, May 12, 13:11

Fabien planned changes to this revision.Mon, May 12, 13:12

Harbormaster completed remote builds in B33275: Diff 53942.Mon, May 12, 13:37

Fabien requested review of this revision.Mon, May 12, 14:29

Fabien edited the summary of this revision. (Show Details)

PiRK accepted this revision.Mon, May 12, 14:30

This revision is now accepted and ready to land.Mon, May 12, 14:30

Closed by commit rABCcdb17840db55: Revert "[chronik] Electrum: add the blockchain.headers.subscribe endpoint" (authored by Fabien). · Explain WhyMon, May 12, 14:40

This revision was automatically updated to reflect the committed changes.

Fabien added a commit: rABCcdb17840db55: Revert "[chronik] Electrum: add the blockchain.headers.subscribe endpoint".

Revision Contents
Changeset List

Path

Size

chronik/

chronik-http/

src/

electrum.rs

104 lines

test/

functional/

chronik_electrum_blockchain.py

64 lines

test_framework/

jsonrpctools.py

42 lines

Diff 53944

View Options

chronik/chronik-http/src/electrum.rs

View Options

test/functional/chronik_electrum_blockchain.py

View Options

test/functional/test_framework/jsonrpctools.py

Revert "[chronik] Electrum: add the blockchain.headers.subscribe endpoint"ClosedPublicActions

Details

Diff Detail

Event Timeline

Revision ContentsChangeset List

Diff 53944

chronik/chronik-http/src/electrum.rs

test/functional/chronik_electrum_blockchain.py

test/functional/test_framework/jsonrpctools.py

Revert "[chronik] Electrum: add the blockchain.headers.subscribe endpoint"
ClosedPublic
Actions

Revision Contents
Changeset List