Skip to content

Fix ETA with endBlock#394

Merged
DZakh merged 1 commit intomainfrom
dz/fix-eta-with-endblock
Dec 23, 2024
Merged

Fix ETA with endBlock#394
DZakh merged 1 commit intomainfrom
dz/fix-eta-with-endblock

Conversation

@DZakh
Copy link
Member

@DZakh DZakh commented Dec 20, 2024

It used to show 2 hours before the fix

image

@DZakh DZakh requested review from JasoonS and moose-code December 20, 2024 16:26

let getTotalRemainingBlocks = (chains: array<ChainData.chainData>) => {
chains->Array.reduce(0, (accum, {progress, currentBlockHeight, latestFetchedBlockNumber}) => {
chains->Array.reduce(0, (accum, {progress, currentBlockHeight, latestFetchedBlockNumber, endBlock}) => {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ok so endBlock is only set if they have it in their config? Makes sense.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes

@DZakh DZakh merged commit 10b9dcd into main Dec 23, 2024
1 check passed
@DZakh DZakh deleted the dz/fix-eta-with-endblock branch December 23, 2024 08:00
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.

2 participants