Skip to content

Conversation

10gic
Copy link
Contributor

@10gic 10gic commented Oct 9, 2025

Add SubscribeTransactionReceipts for ethclient. This is a complement to #32697.

// TransactionReceiptsQuery defines criteria for transaction receipts subscription.
// If TransactionHashes is empty, receipts for all transactions included in new blocks will be delivered.
// Otherwise, only receipts for the specified transactions will be delivered.
type TransactionReceiptsQuery struct {
Copy link
Contributor

Choose a reason for hiding this comment

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

We can duplicate this type in ethclient. Feels weird to have a struct in the interfaces file!

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I see there are other structs in this file, for example, FilterQuery/CallMsg. How should I make the change more appropriately?

Copy link
Contributor

Choose a reason for hiding this comment

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

Fair enough, forgot we have the query types also here.

Copy link
Contributor

@s1na s1na left a comment

Choose a reason for hiding this comment

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

LGTM!

@s1na s1na added this to the 1.16.5 milestone Oct 10, 2025
@s1na s1na changed the title ethclient: add SubscribeTransactionReceipts for ethclient ethclient: add SubscribeTransactionReceipts Oct 10, 2025
@s1na s1na merged commit 659342a into ethereum:master Oct 10, 2025
6 of 7 checks passed
Sahil-4555 pushed a commit to Sahil-4555/go-ethereum that referenced this pull request Oct 12, 2025
Add `SubscribeTransactionReceipts` for ethclient. This is a complement
to ethereum#32697.
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