feat: tictactoe UI - Githubissues

FuelLabs / sway-applications

Swaypplications

https://sway-applications.vercel.app/

Apache License 2.0

1.57k stars 537 forks source link

feat: tictactoe UI #780

Closed matt-user closed 4 months ago

matt-user commented 4 months ago

Type of change

New feature

Changes

The following changes have been made:

Create a ui for the tic tac toe app

Related Issues

Closes #110

vercel[bot] commented 4 months ago

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated (UTC)
sway-applications	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Mar 15, 2024 1:36am

sarahschwartz commented 4 months ago

When trying the Vercel preview I'm getting Error: it is not your turn. Please switch to the correct player. on both accounts.

same

matt-user commented 4 months ago

@K1-R1 @sarahschwartz I think I fixed all the ui errors. After testing the only error I have been encountering is the one discussed with the ts sdk team and at the moment I can't fix that on my end.

matt-user commented 4 months ago

Awesome, it's working great now! Would you mind linking the TS-SDK issue on this PR? Do you think we should wait for the TS-SDK fix before merging? And just as a nice addition in the meantime; is it possible to prompt the account switch between turns, or add a switch account button?

I don't think an issue was created for this bug, but the sdk team said they are looking into it. Looking at the wallet sdk I don't see a way to add a button to switch accounts.