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

Transactional smart contracts #5147

Open
mversic opened this issue Oct 10, 2024 · 0 comments
Open

Transactional smart contracts #5147

mversic opened this issue Oct 10, 2024 · 0 comments
Labels
Refactor Improvement to overall code quality

Comments

@mversic
Copy link
Contributor

mversic commented Oct 10, 2024

Make executing smart contract an instruction instead of part of Executable. I propose we introduce Execute<Wasm> and Execute<Trigger> instead of having ExecuteTrigger instruction

This will take care of some API weirdness because we'll now have Vec<InstructionBox> and will also enable us to combine smart contract execution with other instructions (or smart contracts) in a transaction where all have to succeed together or fail together
@mversic mversic added the Refactor Improvement to overall code quality label Oct 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Refactor Improvement to overall code quality
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@mversic