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

Support complex types in Starknet ABI #1171

Merged
merged 6 commits into from
Dec 19, 2024
Merged

Conversation

m-kus
Copy link
Member

@m-kus m-kus commented Dec 18, 2024

Adds support for struct/array types + special handling for u256 and bytearray;
Fixes config validation (thanks to @droserasprout ); resolves #1172
Fixes missing property in starknet node class
Fixes event parsing (temporary fix while waiting for starknet_py release, see #1142)

@m-kus m-kus force-pushed the fix/cairo-abi-struct branch 2 times, most recently from b41b93f to 1e7067f Compare December 18, 2024 19:03
@m-kus m-kus changed the base branch from next to fix/starknet-events-keys-matcher December 18, 2024 21:43
@m-kus m-kus force-pushed the fix/cairo-abi-struct branch from 89fd3af to 869d18a Compare December 18, 2024 21:44
@m-kus m-kus requested review from droserasprout and Wizard1209 and removed request for droserasprout December 18, 2024 21:54
@m-kus m-kus force-pushed the fix/cairo-abi-struct branch from 869d18a to df7a0a4 Compare December 18, 2024 22:01
@m-kus m-kus changed the base branch from fix/starknet-events-keys-matcher to next December 18, 2024 22:01
@droserasprout droserasprout added bug Something isn't working community Reported or suggested by our awesome users labels Dec 19, 2024
@droserasprout droserasprout modified the milestones: 8.1.3, 8.1.4 Dec 19, 2024
@droserasprout droserasprout merged commit 53b8c2b into next Dec 19, 2024
7 checks passed
@droserasprout droserasprout deleted the fix/cairo-abi-struct branch December 19, 2024 18:56
@droserasprout droserasprout modified the milestones: 8.1.4, 8.1.3 Dec 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working community Reported or suggested by our awesome users
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Config validation failed (starknet template)
3 participants