-
-
Notifications
You must be signed in to change notification settings - Fork 223
feat: generate EIP-7702 gas fee tokens #5706
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
Conversation
a3c3b1a
to
efae3d4
Compare
@metamaskbot publish-preview |
Preview builds have been published. See these instructions for more information about preview builds. Expand for full list of packages and versions.
|
@metamaskbot publish-preview |
Preview builds have been published. See these instructions for more information about preview builds. Expand for full list of packages and versions.
|
71c2f67
to
fd3742a
Compare
@metamaskbot publish-preview |
fd3742a
to
5ce898b
Compare
@metamaskbot publish-preview |
Preview builds have been published. See these instructions for more information about preview builds. Expand for full list of packages and versions.
|
@@ -4234,4 +4230,100 @@ export class TransactionController extends BaseController< | |||
newTransactionMeta, | |||
); | |||
} | |||
|
|||
async #getSimulationData({ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I've an upcoming PR that will better modularise this simulation logic to keep it out of TransactionController.ts
.
Add authorizationList to simulation request.
5ce898b
to
e83ad71
Compare
Explanation
Include
authorizationList
andwith7702
properties in simulation request to return EIP-7702 gas fee tokens, and include upgrade gas.Add optional
isEIP7702GasFeeTokensEnabled
property to constructor.References
Checklist