Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Inject a historical block range #38

Merged
merged 2 commits into from Sep 21, 2022
Merged

Inject a historical block range #38

merged 2 commits into from Sep 21, 2022

Conversation

carver
Copy link
Contributor

@carver carver commented Sep 21, 2022

What was wrong?

FIx #34

How was it fixed?

Change the CLI structure so that you specify a particular command (eg~ --latest to track the head of the chain, or --content-files to inject prebuilt content).

Add a new --block-range flag that lets you specify a historical block range to publish data for.

To-Do

  • Clean up commit history
  • Add section to docs for specifying a block range

Cute Animal Picture

put a cute animal picture link inside the parentheses

@carver carver mentioned this pull request Sep 21, 2022
2 tasks
eth_portal/bridge/__main__.py Outdated Show resolved Hide resolved
eth_portal/bridge/history.py Outdated Show resolved Hide resolved
eth_portal/bridge/history.py Outdated Show resolved Hide resolved
eth_portal/bridge/__main__.py Outdated Show resolved Hide resolved
@carver carver mentioned this pull request Sep 21, 2022
4 tasks
We want more explicit commands given to the bridge module, so that it's
clear whether we are following the latest on the chain, or injecting a
file, or backfilling an old block number.

Update docs with the new CLI argument structure

Add release note mentioning the breaking change to --latest, for
following the head of the chain.
Add docs for backfilling a block range
@carver carver marked this pull request as ready for review September 21, 2022 20:43
@carver carver merged commit ef48cf6 into main Sep 21, 2022
@carver carver changed the title Inject at header Inject a historical block range Sep 21, 2022
@carver carver deleted the inject-at-header branch September 21, 2022 20:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Option to insert data at a specific block
1 participant