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

Cherry-pick #3750: Fix timestamp when querying zkevm_getBatchByNumer and the batch is only in trusted state #3768

Merged
merged 1 commit into from
Aug 14, 2024

Conversation

agnusmor
Copy link
Contributor

What does this PR do?

Cherry picks #3750 into develop

Reviewers

Main reviewers:

@ToniRamirezM

…ly in trusted state (#3750)

* fix timestamp when querying zkevm_getBatchByNumer and the batch is only in trusted state

* control timestamp to be returned for a batch in GetBatchTimestamp

* fix ErrNotFound

* fix state.ErrNotFound check

* fix comments

* fix GetRawBatchTimestamps

* fix comments

* fix GetRawBatchTimestamps return err
@agnusmor agnusmor added cherry-pick Content comes from a previous version branch zkevm-seq-agg labels Aug 14, 2024
@agnusmor agnusmor added this to the v0.8.0 milestone Aug 14, 2024
@agnusmor agnusmor self-assigned this Aug 14, 2024
@cla-bot cla-bot bot added the cla-signed label Aug 14, 2024
@agnusmor agnusmor merged commit 741b9b0 into develop Aug 14, 2024
18 checks passed
@agnusmor agnusmor deleted the cherry-pick/3750-develop branch August 14, 2024 11:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cherry-pick Content comes from a previous version branch cla-signed zkevm-seq-agg
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants