-
Notifications
You must be signed in to change notification settings - Fork 48
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
Implement support for on-chain accounts #142
Comments
For some context: related issues in miden-base: 0xPolygonMiden/miden-base#403 0xPolygonMiden/miden-base#402 |
@bobbinth @hackaugusto @phklive, how do you think, should we add on-chain account details into the response of |
I don't have a great answer to this yet - but maybe this is the right time to start thinking this through. So, a few preliminary thoughts/questions:
One potential approach is as follows:
|
I've implemented one single endpoint for getting the latest details for public account. I think, it's enough for now, but we should create separate issue(s) for storage/endpoints improvements. |
I think this is fine for now - but let's create a separate issue specifically for implementing the delta-based approach. |
Currently, the node can handle only off-chain accounts. To support on-chain accounts we'd need to do the following:
apply_block
.This work requires 0xPolygonMiden/miden-base#403 to be completed.
The text was updated successfully, but these errors were encountered: