-
Notifications
You must be signed in to change notification settings - Fork 16
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] State Update Events #4
Comments
Hi, I'd like to work on this issue! Could you please point me in the right direction to get started; like the first step, and I'll build up from there |
Hey @AryanGodara! Thank you for considering contributing. Currently there's no component for the state. So in this issue you may add the missing event that is linked in the issue description into the app chain contract itself. Does it make sense to you? |
Hey @glihm, I got the issue, can I take it? |
Hi @glihm @drspacemn , any update about this issue? Is anyone working on it? I can do it today if it is available :) |
Hello, I have been working on this with no intention to make a PR initially but it turns out I can make one soon. |
Implement and test the State Update Events.
Can mock or hard code a successful state update for the time being.
References
https://github.com/starkware-libs/cairo-lang/blob/ab6d079f0265f903630aac9287e93d9e7cfb6fcf/src/starkware/starknet/solidity/Starknet.sol#L216
https://github.com/starkware-libs/cairo-lang/blob/ab6d079f0265f903630aac9287e93d9e7cfb6fcf/src/starkware/starknet/solidity/Starknet.sol#L248
The text was updated successfully, but these errors were encountered: