[Examples/Front-end] Tic-tac-toe #18526
Merged
Commits on Jul 11, 2024
-
## Description Port Move code for the tic tac toe example into the `sui/examples` directory, modernising it in the process: - Standardise the implementation between the owned, shared and multi-sig variants, so that the only differences are relevant to what they need to do differently. In doing this, the owned and multi-sig variants become identical (at least at the level of Move code). - Use transfer-to-object in the "owned" example, to send potential moves to the `Game` object, instead of the admin that owns the game. - Use more Move 2024 features (`syntax(index)`, receiver function aliases, etc). - Improve test coverage ## Test plan New unit tests: ``` tic_tac_toe/move$ sui move test ```
-
-
-
-
-
[Examples/Tic-Tac-Toe] Front-end
## Description Build front-ends for shared and multi-sig tic-tac-toe, using `create-react-dapp`. ## Test plan Manually tested, (screenshots TBD).
-
-
-
-
-
fixup: pnpm workspace unignore tic-tac-toe
Use --ignore-workspace when installing instead.
-
-
-
-
fixup: Use Polymedia for local explorer
Couldn't get SuiScan's local explorer to work.
-
-
-
-
This will change for each person, so don't check-in an actual package.
-
fixup: Address some more comments.
- useExecutor exposes more info about mutation status, we leverage this for the play buttons. - Early return style for validation labels. - Strict equality for address validation. - `<>` instead of `<div>`
-
-
-
-
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.