Skip to content
This repository has been archived by the owner on Aug 21, 2024. It is now read-only.

ExecutableTransaction should be implemented on every tx type #1709

Open
tdelabro opened this issue Mar 22, 2024 · 0 comments
Open

ExecutableTransaction should be implemented on every tx type #1709

tdelabro opened this issue Mar 22, 2024 · 0 comments

Comments

@tdelabro
Copy link
Contributor

Right now ExecutableTransaction is only implemented on Transaction, AccountTransaction and L1HandlerTransaction.

It forces me to wrap my InvokeTransaction in an AccoutTransaction to execute it.

This has no reason to be the case, ExecutableTransaction should be implemented on InvokeTransaction as well as DeclareTransaction and DeployAccountTransaction.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@tdelabro