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

feat(store): persist zkapp actions in db #1712

Merged
merged 13 commits into from
Jan 24, 2025
Merged

feat(store): persist zkapp actions in db #1712

merged 13 commits into from
Jan 24, 2025

Conversation

Isaac-DeFrain
Copy link
Member

Describe your changes

  • adds a ZkappActionStore trait
  • implements the ZkappActionStore trait for IndexerStore
  • adds unit tests for ZkappActionStore functionality
  • persists actions in the best ledger store

Link issue(s) fixed

Checklist before requesting a review

  • I have performed a self-review of my code.
  • I have added tests that prove my fix is effective or that my feature works.
  • I have verified new and existing tests pass locally with my changes.
  • I verified whether it was necessary to increment the database version.

@Isaac-DeFrain Isaac-DeFrain linked an issue Jan 23, 2025 that may be closed by this pull request
Store actions in the db #1705
Closed
@Isaac-DeFrain Isaac-DeFrain mentioned this pull request Jan 23, 2025
Merged
4 tasks
@Isaac-DeFrain
Copy link
Member Author

Merge before #1713

@jhult jhult merged commit 7cedd0b into Granola-Team:main Jan 24, 2025
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jhult jhult jhult approved these changes

@robinbb robinbb Awaiting requested review from robinbb robinbb is a code owner

Assignees

@jhult jhult

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Store actions in the db
2 participants
@Isaac-DeFrain @jhult