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

new tutorial 'Introduction to write Smart Contracts in Rust and Ink! #11

Merged
merged 3 commits into from
Feb 1, 2024

Conversation

GuiGou12358
Copy link
Contributor

Add a new tutorial Introduction to write Smart Contracts in Rust and Ink!

Copy link
Contributor

@meganskye meganskye left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

you should ensure any errors thrown during a test build are cleared up before submitting a PR.

file and folder names shouldn't contain characters that need to be escaped. for prettier styling of docusaurus document and section titles, use:

---
title: Pretty Name
---

and/or _category_.json files.

@GuiGou12358
Copy link
Contributor Author

@meganskye The file has been renamed and the build is fine now.

Rename files using - instead of spaces;
Modifying images to use the figure tag;
Copy link
Collaborator

@Gunit2481 Gunit2481 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link
Contributor

@meganskye meganskye left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

Copy link
Contributor Author

@GuiGou12358 GuiGou12358 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Agree with the changes done by @Gunit2481

@Gunit2481 Gunit2481 merged commit 1045807 into AstarNetwork:main Feb 1, 2024
1 check passed
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.

3 participants