[chronik.e.cash] Update chronik-client in docs
ClosedPublic
Actions

Authored by bytesofman on Sep 11 2024, 20:55.

Details

Reviewers

emack

Group Reviewers

Restricted Project

Commits

rABC000f5d048bb4: [chronik.e.cash] Update chronik-client in docs

Summary

Update docs page to always use the latest version of chronik-client

Test Plan

docker build -f chronik.e.cash.Dockerfile -t chronik-docs .

Diff Detail

Repository

rABC Bitcoin ABC

Branch

cc-docs-update-no-deps

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 30341
Build 60208: Build Diff
Build 60207: arc lint + arc unit

Event Timeline

bytesofman created this revision.Sep 11 2024, 20:55

Herald added a reviewer: Restricted Project. · View Herald TranscriptSep 11 2024, 20:55

Harbormaster completed remote builds in B30262: Diff 49601.Sep 11 2024, 20:55

do not use relative installs for deps

Harbormaster completed remote builds in B30283: Diff 49627.Sep 13 2024, 22:26

bytesofman retitled this revision from [chronik.e.cash] Use relative import for chronik-client to [chronik.e.cash] Update chronik-client in docs.Sep 13 2024, 22:33

bytesofman published this revision for review.Sep 18 2024, 04:27

accepted with minor nits

web/chronik.e.cash/docs/1-chronik-js/01-connect.mdx
9 ↗	(On Diff #49627)
13 ↗	(On Diff #49627)
web/chronik.e.cash/docs/1-chronik-js/05-addresses.mdx
65 ↗	(On Diff #49627)

This revision is now accepted and ready to land.Sep 18 2024, 05:52

bytesofman mentioned this in D16699: Update chronik documentation after NNG deprecation.Sep 18 2024, 12:45

bytesofman marked 3 inline comments as done.Sep 18 2024, 13:10

bytesofman added inline comments.

web/chronik.e.cash/docs/1-chronik-js/05-addresses.mdx
65 ↗	(On Diff #49627)	max page size is 200, so while it is weird, this kind of thing does make sense (esp for an addr with a lot of history like the IFP address) we're asking for "page 300 if we only show 2 txs per page"