Skip to content

ethclient: fix BlobSidecars api#2656

Merged
zzzckck merged 1 commit intobnb-chain:developfrom
jhgdike:dike-fix
Aug 22, 2024
Merged

ethclient: fix BlobSidecars api#2656
zzzckck merged 1 commit intobnb-chain:developfrom
jhgdike:dike-fix

Conversation

@jhgdike
Copy link
Copy Markdown
Contributor

@jhgdike jhgdike commented Aug 20, 2024

Description

Fix the APIs response of BlobSidecars and BlobSidecarByTxHash

Rationale

Recently, I used the APIs above but they didn't work, and I found the response has the wrong structure. So I fixed it.

Copy link
Copy Markdown
Contributor

@galaio galaio left a comment

Choose a reason for hiding this comment

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

LGTM

@zzzckck zzzckck merged commit c46d7e8 into bnb-chain:develop Aug 22, 2024
@jhgdike jhgdike deleted the dike-fix branch August 22, 2024 09:57
// BlobSidecarByTxHash return a sidecar of a given blob transaction
func (ec *Client) BlobSidecarByTxHash(ctx context.Context, hash common.Hash) (*types.BlobTxSidecar, error) {
var r *types.BlobTxSidecar
func (ec *Client) BlobSidecarByTxHash(ctx context.Context, hash common.Hash) (*types.BlobSidecar, error) {
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

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.

4 participants