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

Update starknet foundry sncast docs #218

Merged
merged 1 commit into from
Oct 15, 2023

Conversation

dalmasonto
Copy link
Contributor

I have create a documentation on how to use sncast extensively in order to interact with contracts on starknet devnet. check it out and if there are any updates, let me know.

@omarespejel omarespejel merged commit d48a3e6 into starknet-edu:main Oct 15, 2023
@omarespejel
Copy link
Collaborator

This is amazing @dalmasonto! Thank you! I merged it and will review it right now. I will come back later today if there is any feedback so we can release the Only Dust award. Thanks!

@omarespejel
Copy link
Collaborator

Hey @dalmasonto, great job! I've just merged an edit that refines the format to align it better with the rest of the book. Don't worry; this is a standard part of the process. You can check it out here: [Link to the edited page](https://book.starknet.io/ch02-12-foundry-cast.html). Your work is fantastic; thank you for your contribution!

Would you mind adding another example in the same Cast subchapter where you create a deployment and interaction script for the same smart contract you've already used? For instance, you can create a sh file that tests, declares, deploys, and makes a multicall to the contract. I've discussed this with the Foundry team, and it's a common use case for Cast. You can even utilize the calls you've defined earlier; it's mainly a matter of creating the sh file and demonstrating how to run it. What are your thoughts on this?

@dalmasonto
Copy link
Contributor Author

Thank you. Yea, will write a script that does the same also. Once done, I will update you

@omarespejel
Copy link
Collaborator

thank you @dalmasonto, it is already looking amazing!

@dalmasonto
Copy link
Contributor Author

Hello @omarespejel, just created a PR for the script. It might not be the best but check it out

@dalmasonto
Copy link
Contributor Author

@omarespejel I have updated the PR here #220 to make sure that the markdown is properly formatted

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.

None yet

2 participants