-
Notifications
You must be signed in to change notification settings - Fork 31
Feat/claim tx without operator complete P2P #385
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
Conversation
ltardivo
left a comment
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.
Looks good to me, we can add the description of the PR and maybe rename it, since here we are removing the chess-server
ClemensLey
left a comment
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 work!
|
When the white player creates the game link, to be sent to the black player, we should update the text in the modal as follows: "Share this link with your opponent to start the game." -> "Share this link with the black player. The black player will get back to you with the final game link." |
|
The sign-in button doesn't work the second time |
|
update the message to ask user (add link to my-games page) to release fund from the /users page if the release-fund action failed |
| {data.winnerPubKey === data.userPubKey | ||
| ? `Congratiolations! You have won the game. ` | ||
| ? `Congratulations! You have won the game. ` | ||
| : `Sorry! You have lost the game. `} |
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.
- When a player lose the game, we should invite him/her to the
my-gamespage to release the funds. - Also, we need a way to go to the
my-gamespage, maybe from the left panel or from any place you think.
| requestRelease, | ||
| releaseFund, | ||
| ) | ||
| // !game ? ( |
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.
Is this commented code not needed anymore?
| ? `${chessContract.satoshis / BigInt(1e8)} ${computer.getChain()}` | ||
| : 'Loading...'} | ||
| </td> | ||
| <td>{game ? getGameState(game) : 'Loading...'}</td> |
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.
I got a typescript error here:
src/components/MyGames/index.tsx:157:32 - error TS2345: Argument of type 'Chess' is not assignable to parameter of type 'ChessContract'.
Type 'Chess' is missing the following properties from type 'ChessContract': satoshis, nameW, nameB, publicKeyW, and 10 more.
157 <td>{game ? getGameState(game) : 'Loading...'}</td>
~~~~
````
Chess App that's complete P2P
Typescript upgrade which fixes the long term related to Lib build
Removed chess server