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

Thiagodeev/rpcv08 final changes #644

Open
wants to merge 7 commits into
base: v0.8.0
Choose a base branch
from

Conversation

thiagodeev
Copy link
Collaborator

@thiagodeev thiagodeev commented Nov 6, 2024

#623

This PR aims to implement some RPC v0.8.0 updates to starknet.go. The ones addressed by this PR are:

Components
CONTRACT_EXECUTION_ERROR
EXECUTION_RESOURCES
COMPUTATION_RESOURCES (remotion)

Errors
CONTRACT_ERROR
TRANSACTION_EXECUTION_ERROR

Note: As the RPC v0.8.0 has not yet been released, we can't fully test these changes as the nodes haven't implemented them yet. They will be updated before merging into main

Also, it improves the RPCError type. Now we have a way of returning the error message contained in the Data error field. We can also access it just by checking if it's nil or not.

@thiagodeev thiagodeev force-pushed the thiagodeev/rpcv08-finalChanges branch from 4108a47 to 629890b Compare November 6, 2024 18:54
@thiagodeev thiagodeev force-pushed the thiagodeev/rpcv08-finalChanges branch from 629890b to b7ce944 Compare November 7, 2024 12:59
@thiagodeev thiagodeev marked this pull request as ready for review November 8, 2024 02:31
@rianhughes rianhughes self-requested a review November 8, 2024 09:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant