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

Fix typed transaction hashing #1144

Merged
merged 4 commits into from
Jan 5, 2022
Merged

Fix typed transaction hashing #1144

merged 4 commits into from
Jan 5, 2022

Conversation

tgmichel
Copy link
Contributor

@tgmichel tgmichel commented Jan 5, 2022

Update the locally calculated transaction hashes to support new transaction types.

@tgmichel tgmichel added A0-pleasereview Pull request needs code review. B7-runtimenoteworthy Changes should be noted in any runtime-upgrade release notes labels Jan 5, 2022
@tgmichel tgmichel merged commit 2967257 into master Jan 5, 2022
@tgmichel tgmichel deleted the tgm-tracing-test-fix branch January 5, 2022 16:25
@boundless-forest
Copy link
Contributor

I guess this pr should also carry the A10-evmtracing label? @librelois

@librelois librelois added A10-evmtracing Pull request includes evm tracing functionality and removed A0-pleasereview Pull request needs code review. labels Jan 6, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@librelois librelois librelois approved these changes

Assignees
No one assigned
Labels
A10-evmtracing Pull request includes evm tracing functionality B7-runtimenoteworthy Changes should be noted in any runtime-upgrade release notes
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@tgmichel @boundless-forest @librelois