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

Add TicTacToe tutorial doc #836

Merged
merged 2 commits into from
Mar 15, 2022
Merged

Add TicTacToe tutorial doc #836

merged 2 commits into from
Mar 15, 2022

Conversation

lxfind
Copy link
Contributor

@lxfind lxfind commented Mar 15, 2022

This is not linked from anywhere yet.
We can decide where to link to this latter.

Copy link
Collaborator

@gdanezis gdanezis left a comment

Choose a reason for hiding this comment

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

Thanks for this -- love it!

doc/src/explore/tutorials.md Outdated Show resolved Hide resolved
doc/src/explore/tutorials.md Outdated Show resolved Hide resolved
In this tutorial, we will demonstrate an end-to-end process of starting a Sui network locally, collecting to it through our demo wallet app, publish a TicTacToe game on Sui and play it to the end.

## Start the network
In the Sui repository, build Sui:
Copy link
Contributor

Choose a reason for hiding this comment

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

I think the plan is to have a single "Install" section, but since it does not exist yet, I'd say we should land this one "as is" and adjust this later.

@lxfind lxfind force-pushed the structured-publish-response branch from 597e081 to 63b92d3 Compare March 15, 2022 18:16
Base automatically changed from structured-publish-response to main March 15, 2022 18:47
@lxfind lxfind merged commit 6b6aae8 into main Mar 15, 2022
@lxfind lxfind deleted the tictactoe-tutorial branch March 15, 2022 18:50
@Clay-Mysten
Copy link
Contributor

Did Adam's input get addressed? Is it worth me doing an edit if not?

@lxfind
Copy link
Contributor Author

lxfind commented Mar 15, 2022

The last comment was not addressed, that we should also have an installation doc somewhere

@lxfind
Copy link
Contributor Author

lxfind commented Mar 15, 2022

I also have a pending update: #848

mwtian pushed a commit that referenced this pull request Sep 12, 2022
Bumps [either](https://github.com/bluss/either) from 1.7.0 to 1.8.0.
- [Release notes](https://github.com/bluss/either/releases)
- [Commits](rayon-rs/either@1.7.0...1.8.0)

---
updated-dependencies:
- dependency-name: either
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
mwtian pushed a commit to mwtian/sui that referenced this pull request Sep 29, 2022
Bumps [either](https://github.com/bluss/either) from 1.7.0 to 1.8.0.
- [Release notes](https://github.com/bluss/either/releases)
- [Commits](rayon-rs/either@1.7.0...1.8.0)

---
updated-dependencies:
- dependency-name: either
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
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

4 participants