Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[Feature] Add isthmus operator fee fields to rpc receipt #419

Closed
mattsse opened this issue Feb 6, 2025 · 1 comment · Fixed by #420
Closed

[Feature] Add isthmus operator fee fields to rpc receipt #419

mattsse opened this issue Feb 6, 2025 · 1 comment · Fixed by #420
Labels
good-first-issue Issues that are easy to pick up for new contributors

Comments

@mattsse
Copy link
Member

mattsse commented Feb 6, 2025

Component

rpc-types

Describe the feature you would like

ref:

paradigmxyz/reth#14243

fyi @leruaa

Additional context

No response
@mattsse mattsse added the good-first-issue Issues that are easy to pick up for new contributors label Feb 6, 2025
@leruaa
Copy link
Contributor

leruaa commented Feb 6, 2025 •
edited
Loading

Done in this PR: #130

In fact it's not the same thing

@leruaa leruaa mentioned this issue Feb 6, 2025
Merged
3 tasks
github-merge-queue bot pushed a commit that referenced this issue Feb 6, 2025
@leruaa
Add operator fee to rpc l1 block (#420)
5f764f6 
<!--
Thank you for your Pull Request. Please provide a description above and
review
the requirements below.

Bug fixes and new features should include tests.

Contributors guide:
https://github.com/alloy-rs/core/blob/main/CONTRIBUTING.md

The contributors guide includes instructions for running rustfmt and
building the
documentation.
-->

<!-- ** Please select "Allow edits from maintainers" in the PR Options
** -->

## Motivation

We need to show operator fee new params in RPC receipts.

Close #419
Ref paradigmxyz/reth#14243

<!--
Explain the context and why you're making that change. What is the
problem
you're trying to solve? In some cases there is not a problem and this
can be
thought of as being the motivation for your change.
-->

## Solution

<!--
Summarize the solution and provide any necessary context needed to
understand
the code change.
-->

## PR Checklist

- [ ] Added Tests
- [ ] Added Documentation
- [ ] Breaking changes
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
good-first-issue Issues that are easy to pick up for new contributors
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Add operator fee to rpc l1 block leruaa/op-alloy
2 participants
@leruaa @mattsse