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.

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

feat(dan): add WASM template invocation from user instruction #4331

Merged

Conversation

sdbondi
Copy link
Member

@sdbondi sdbondi commented Jul 21, 2022

Description

Adds minimum required code to call a WASM template from a user instruction.

Motivation and Context

This PR adds and stubs out the required code to call WASM templates from a user instruction. This PR makes use of radix's excellent sbor encoding library, but this is not required and could be switched out.

It's quite messy and incomplete in places.

How Has This Been Tested?

Adds an integration test that compiles and calls the stubbed-out template code.
This is bordering on POC code, so no other tests are written - tests should be implemented after some feedback/iterations from this PR.

@sdbondi sdbondi changed the title feat(dan): add basic WASM template invocation feat(dan): add WASM template invocation from user instruction Jul 21, 2022
@sdbondi sdbondi force-pushed the dan-layer-smart-contract-initial branch 7 times, most recently from 4ff1b4b to ee89dcb Compare July 22, 2022 12:55
@sdbondi sdbondi force-pushed the dan-layer-smart-contract-initial branch from ee89dcb to b64fadc Compare July 22, 2022 13:37
@aviator-app aviator-app bot merged commit d265c6f into tari-project:development Jul 22, 2022
@sdbondi sdbondi deleted the dan-layer-smart-contract-initial branch July 22, 2022 14:25
sdbondi added a commit to sdbondi/tari that referenced this pull request Jul 22, 2022
* development:
  feat(dan): add WASM template invocation from user instruction (tari-project#4331)
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.

2 participants