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

Refactor transaction kernel events and errors #768

Open
Dominik1999 opened this issue Jun 24, 2024 · 0 comments
Open

Refactor transaction kernel events and errors #768

Dominik1999 opened this issue Jun 24, 2024 · 0 comments
Assignees
@Dominik1999
Milestone
v0.5

Comments

@Dominik1999
Copy link
Collaborator

Dominik1999 commented Jun 24, 2024 •
edited by bobbinth
Loading

We need to refactor the kernel a bit in how we handle storage-related procedures.

  • Procedures in api.masm handle:
    • Authentication checks (as they do now).
    • Emitting events.
  • Procedures in the subsequent files, like kernels/tx/account.masm handle:
    • Slot type checks.
    • The actual update of the storage.
  • For get/set_account_item and get/set_account_map_item we should have corresponding procedures kernels/tx/account.masm.

If we don't do this in this PR, let's create an issue for it.

Originally posted by @bobbinth in #758 (comment)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Dominik1999 Dominik1999

Labels
None yet
Projects
User's testnet
Status: In Progress
Milestone
v0.5
Development

No branches or pull requests
2 participants
@Dominik1999 @bobbinth