Skip to content

Conversation

@thepabloaguilar
Copy link
Contributor

Fixes #71

Motivation

Implementing Mempool endpoint.

Solution

To get the transactions in "memory" I've used the txpool_content method which returns all pending and queued transactions.
More about on https://geth.ethereum.org/docs/rpc/ns-txpool#txpool_content

@thepabloaguilar
Copy link
Contributor Author

@shrimalmadhur could you review this PR if possible, please?

@thepabloaguilar
Copy link
Contributor Author

@shrimalmadhur do I need to do something more to merge? Btw, I don't have access to the merge button

@shrimalmadhur
Copy link
Contributor

@shrimalmadhur do I need to do something more to merge? Btw, I don't have access to the merge button

@thepabloaguilar there is problem on our side. I am trying to fix it. You don't have to do anything. will circle back once I fix it.

@thepabloaguilar
Copy link
Contributor Author

Ok, thanks a lot @shrimalmadhur

@shrimalmadhur shrimalmadhur merged commit 4f29f57 into coinbase:master Jan 24, 2022
@shrimalmadhur
Copy link
Contributor

@thepabloaguilar merged it now. apologies for the delay. we had some internal issues with approval.

@thepabloaguilar thepabloaguilar deleted the issue-71 branch January 25, 2022 02:18
@thepabloaguilar
Copy link
Contributor Author

No worries @shrimalmadhur, thanks for the support! Btw, there's another approved PR of mine #87

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

Implement mempool endpoint

2 participants