Skip to content

Conversation

ligi
Copy link
Member

@ligi ligi commented Nov 16, 2022

closes #26188

Copy link
Contributor

@holiman holiman left a comment

Choose a reason for hiding this comment

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

LGTM

@holiman holiman changed the title ethclient: Fix misleading documentation ethclient: docs, fix misleading comment Nov 16, 2022
@holiman holiman added this to the 1.11.0 milestone Nov 16, 2022
@holiman holiman merged commit bc90a88 into ethereum:master Nov 16, 2022
shekhirin pushed a commit to shekhirin/go-ethereum that referenced this pull request Jun 6, 2023
gzliudan added a commit to gzliudan/XDPoSChain that referenced this pull request Aug 22, 2025
gzliudan added a commit to XinFinOrg/XDPoSChain that referenced this pull request Aug 22, 2025
…lter ethereum#25186 ethereum#26189 (#1372)

* eth/filters, ethclient/gethclient: add fullTx option to pending tx filter ethereum#25186

This PR adds a way to subscribe to the _full_ pending transactions, as opposed to just being notified about hashes.

In use cases where client subscribes to newPendingTransactions and gets txhashes only to then request the actual transaction, the caller can now shortcut that flow and obtain the transactions directly.

Co-authored-by: Felix Lange <fjl@twurst.com>

* ethclient: docs, fix misleading comment ethereum#26189

---------

Co-authored-by: lmittmann <lmittmann@users.noreply.github.com>
Co-authored-by: Felix Lange <fjl@twurst.com>
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.

docs: network ID is not chain ID
2 participants