-
Notifications
You must be signed in to change notification settings - Fork 200
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
Fix: several typos in the documentation #929
Fix: several typos in the documentation #929
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great comments! Thank you so much :)
Please correct the few things I mentioned, and then it's good to go.
If you prefer that I take care of it, just let me know.
...nents/Starknet/modules/architecture_and_concepts/pages/Smart_Contracts/cairo-and-sierra.adoc
Outdated
Show resolved
Hide resolved
components/Starknet/modules/architecture_and_concepts/pages/Smart_Contracts/contract-abi.adoc
Outdated
Show resolved
Hide resolved
@@ -70,7 +70,7 @@ When working with the CLI, you can manually set the endpoints for the gateways t | |||
interact with Starknet, by including the following options: | |||
|
|||
`--feeder_gateway_url`:: Sets the custom endpoint for read commands. | |||
`--gateway_url`:: Sets the custom endpoint for write commands. | |||
`--gateway_url`:: Sets the custom endpoint for writing commands. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This change is incorrect. The sentence is talking about read/write commands.
components/Starknet/modules/quick_start/pages/declare_a_smart_contract.adoc
Outdated
Show resolved
Hide resolved
Co-authored-by: Steve Goodman <[email protected]>
Co-authored-by: Steve Goodman <[email protected]>
Co-authored-by: Steve Goodman <[email protected]>
Co-authored-by: Steve Goodman <[email protected]>
hello @stoobie i didn t see the notification 3 days ago, check now please. Thanks ! |
components/Starknet/modules/architecture_and_concepts/pages/Smart_Contracts/contract-abi.adoc
Outdated
Show resolved
Hide resolved
…art_Contracts/contract-abi.adoc
2553f91
into
starknet-io:tudorpintea999/fix-typo
Merged to new branch to generate preview before merging to dev. |
* Fix: several typos in the documentation (#929) * Update cairo-and-sierra.adoc * Update contract-abi.adoc * Update contract-address.adoc * Update contract-classes.adoc * Update contract-storage.adoc * Update hash-functions.adoc * Update starknet-state.adoc * Update starkli.adoc * Update declare_a_smart_contract.adoc * Update juno_versions.adoc * Update pathfinder_versions.adoc * fix typo doc_guidelines.adoc * fix contract-abi.adoc Co-authored-by: Steve Goodman <[email protected]> * fix starkli.adoc Co-authored-by: Steve Goodman <[email protected]> * fix declare_a_smart_contract.adoc Co-authored-by: Steve Goodman <[email protected]> * fix doc_guidelines.adoc Co-authored-by: Steve Goodman <[email protected]> * Update starkli.adoc * Update cairo-and-sierra.adoc * Update components/Starknet/modules/architecture_and_concepts/pages/Smart_Contracts/contract-abi.adoc --------- Co-authored-by: Steve Goodman <[email protected]> * Apply suggestions from code review --------- Co-authored-by: iwantanode <[email protected]>
Description of the Changes
Hello
I fixed some typos in the docs,
Hope it helps.
Br,
Tudor.
This change is