Skip to content

L2: After integration, replace the deposit txs magic data hack with our own deposit/privileged transaction type. #2147

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.

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

Closed
Tracked by #2142
Arkenan opened this issue Mar 5, 2025 · 0 comments · Fixed by #2332

Comments

@Arkenan
Copy link
Collaborator

Arkenan commented Mar 5, 2025

No description provided.

@Arkenan Arkenan changed the title After integration, replace the deposit txs magic data hack with our own deposit/privileged transaction type. L2: After integration, replace the deposit txs magic data hack with our own deposit/privileged transaction type. Mar 5, 2025
github-merge-queue bot pushed a commit that referenced this issue Apr 15, 2025
**Motivation**

In this PR we refactor some of the code and add functionality to work as
expected. Remove the txs magic hack data #2147

**Description**

* Add a `recipient` field to the privileged transactions.
* Deposit the value to the recipient instead of the `to` address.
* Add functionality to call a contract and make a deposit in the same
privileged transaction.
* Remove the signing of the privileged transactions.
* Remove the checking of the nonce and balance when making the deposit,
here we mint the tokens.
* Add new hook in the L2 to address this new features for the Privileged
Transactions.

Closes #2147

---------

Co-authored-by: fborello-lambda <[email protected]>
Co-authored-by: Federico Borello <[email protected]>
Co-authored-by: Estéfano Bargas <[email protected]>
Co-authored-by: Javier Chatruc <[email protected]>
Co-authored-by: Manuel Iñaki Bilbao <[email protected]>
Co-authored-by: Tomás Casagrande <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

1 participant