ethereumjs-monorepo icon indicating copy to clipboard operation
ethereumjs-monorepo copied to clipboard

RPC method eth_feeHistory implementation

Open GitMark0 opened this issue 2 years ago • 5 comments

Implements: eth_feeHistory method for tracking historical gas information

Method is used as described in: https://docs.infura.io/networks/ethereum/json-rpc-methods/eth_feehistory

GitMark0 avatar Jun 11 '23 16:06 GitMark0

Hi @GitMark0, hope we didn't push you away here, but remarks like the one from @jochem-brouwer are simply part of our normal review process we also apply internally? 😬 🙂

Would it be possible to get this over the finish line? This would be somewhat of a pity otherwise, there is already so much great and extensive work in the PR here!

holgerd77 avatar Jul 31 '23 17:07 holgerd77

Hi @GitMark0, hope we didn't push you away here, but remarks like the one from @jochem-brouwer are simply part of our normal review process we also apply internally? 😬 🙂

Would it be possible to get this over the finish line? This would be somewhat of a pity otherwise, there is already so much great and extensive work in the PR here!

Hey @holgerd77 thanks for reaching out! Unfortunately I just didn't find time to finish the PR. I got really busy with a new (unplanned) project a day or two after I opened it... I'd love to finish the PR and continue contributing to the EthereumJS community and I hope that I'll have time to do it by the end of the month. Cheers!

GitMark0 avatar Aug 01 '23 08:08 GitMark0

Cool, no problem and no rush! 🙂👍🙏

holgerd77 avatar Aug 01 '23 08:08 holgerd77

@GitMark0 Hi Berry, I would assume you are fully caught in with your new duties, right? 🙂 Thanks for starting this great PR! We (respectively @jochem-brouwer from our team) would take this over now, we would cherry-pick your commits into a new PR (so that you retain ownership) and will the give this some finalization.

For now I would assume this is ok for you, let us know if you have reservations or some plain rejection here though!

All the best Holger

holgerd77 avatar Feb 20 '24 10:02 holgerd77

Follow-up: #3295

jochem-brouwer avatar Feb 27 '24 11:02 jochem-brouwer