Ensure starknet_call calldata is not null #658
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
thiagodeev
requested changes
Jan 22, 2025
thiagodeev
approved these changes
Jan 23, 2025
thiagodeev
added a commit
that referenced
this pull request
Jan 30, 2025
commit f7015b7 Merge: 9199306 a688cba Author: Thiago Ribeiro <[email protected]> Date: Thu Jan 30 01:26:59 2025 -0300 Merge branch 'main' into v0.8.0 commit a688cba Author: Franco Barpp Gomes <[email protected]> Date: Fri Jan 24 02:19:12 2025 +0900 Ensure starknet_call calldata is not null (#658) * fix: Fix function call calldata possibly nil * refactor: Remove empty calldata package var commit 46011b2 Author: Thiago Ribeiro <[email protected]> Date: Tue Jan 21 06:43:33 2025 -0300 Update README.md to include typedData example (#653) commit 45554f0 Author: Olexandr88 <[email protected]> Date: Mon Jan 13 16:46:46 2025 +0200 docs: changed the link to shield stars (#652) Update README.md
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Refers to #657. Feel free to merge this if it makes sense, and not otherwise!