Page MenuHomePhabricator

[explorer] Switch to new chronik
ClosedPublic

Authored by Fabien on Tue, Apr 9, 15:21.

Details

Reviewers
PiRK
tobias_ruck
Group Reviewers
Restricted Project
Commits
rABC42f15d7e8a7e: [explorer] Switch to new chronik
Summary

This diff contains the last steps that cannot be split apart to migrate the explorer to the in-node chronik:

  • Update the pinned version of the dependencies so it supports the in-node chronik
  • Update block txs retrieval via the new block-txs endpoint
  • Accommodate the new API/Protobuf

The token parsing is assuming a single action per token and only the first one (if any) is displayed. This can be improved in follow ups.
This diff is only targeting feature parity with the NNG version.

Depends on D15962.

Test Plan
cargo run

Diff Detail

Repository
rABC Bitcoin ABC
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

Feature parity with NNG excepted raw block header and nonce

Remove debug comments, pin bitcoinsuite depdendencies version.

TODO: Properly handle tokens (including Mint Vault and ALP)

Rebase, fix token_id == 0, display SLPv2 and ALP details (still assuming a single token action per tx, which is wrong for ALP)

Fabien published this revision for review.Fri, Apr 12, 13:38
Fabien edited the summary of this revision. (Show Details)
This revision is now accepted and ready to land.Fri, Apr 12, 14:38
tobias_ruck added a subscriber: tobias_ruck.
tobias_ruck added inline comments.
web/explorer/explorer-server/code/txs.js
194 ↗(On Diff #47090)

Nit: this should be camel case, welcome to JS

This revision now requires changes to proceed.Fri, Apr 12, 14:59

CamelCasify the snake_casified js function

This revision is now accepted and ready to land.Fri, Apr 12, 16:11
This revision was automatically updated to reflect the committed changes.