Skip to content

refactor: replacing use of L2Tx with TxEffect - #4858

Closed
benesjan wants to merge 6 commits into
masterfrom
02-29-refactor_replacing_use_of_L2Tx_with_TxEffect
Closed

benesjan wants to merge 6 commits into
masterfrom
02-29-refactor_replacing_use_of_L2Tx_with_TxEffect

Conversation

@benesjan

Copy link
Copy Markdown
Contributor

Fixes #4845

benesjan commented Feb 29, 2024 •

Copy link
Copy Markdown
Contributor Author

@benesjan benesjan changed the title refactor: replacing use of L2Tx with TxEffect refactor: replacing use of L2Tx with TxEffect Feb 29, 2024

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Moved the implementation to aztec node since I needed an access to block source to be able to populated the block number and block hash of the receipt.

@benesjan
benesjan force-pushed the 02-29-refactor_replacing_use_of_L2Tx_with_TxEffect branch from 5d9f7fd to 28fadab Compare February 29, 2024 15:04
@benesjan
benesjan force-pushed the 02-29-refactor_replacing_use_of_L2Tx_with_TxEffect branch from 88dc0c8 to 29348d6 Compare February 29, 2024 16:04
@benesjan

benesjan commented Mar 1, 2024

Copy link
Copy Markdown
Contributor Author

Can't push to this branch anymore (probably because graphite expired). So I am opening a new branch #4876

@benesjan benesjan closed this Mar 1, 2024
@ludamad
ludamad deleted the 02-29-refactor_replacing_use_of_L2Tx_with_TxEffect branch August 22, 2024 14:18
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.

Replace usage of L2Tx with TxEffect and nuke it

1 participant