code100x
Feat: Added `Timing for each move`, `Spectator Feature`, `Chat Feature` , `Review Page` and updated the ui for it
Closes #148 #142 #225 #226 #227 #316
- Each move Timing in the Table
- Updated the logic for User timer as it was delayed in prefix PR(Now Fixed)
- SPECTATOR feature
- Chat Feature
- Review Feature - you can check the moves
Updated with all the latest Changes and solved all merge Conflicts
https://github.com/code100x/chess/assets/95279849/0cc0638d-4d7e-485e-9e72-efeffc7a13b0
@hkirat Is this good?
changed some of the logic in your last PR Mind getting rid of the conflicts?
changed some of the logic in your last PR Mind getting rid of the conflicts?
Okay I will update it with the new one
@hkirat Added Spectator Feature
https://github.com/code100x/chess/assets/95279849/2c8089c5-0398-476f-aa50-e5abf1e0e84e
@hkirat
Added Review Page where users can see all the moves and table
https://github.com/code100x/chess/assets/95279849/2d90953f-8e5e-4939-9306-a1b1d96ff3bf
@hkirat please check it it was a tough task to resolve the conflicts and commit once again Thanks
@Airbornharsh can you update the recordings after the latest changes
Yeah okay
@hkirat @nimit9
https://github.com/code100x/chess/assets/95279849/0e24800d-663e-48bf-9183-4dbb4d39eafc
Check it out?
@Airbornharsh please remove package-lock from root, and only use yarn on local.
Also, since you haven't modified the package.json, yarn-lock should be deleted @Airbornharsh
doesnt have the latest lock file @Airbornharsh
Updated
dont know why it's showing a whitespace diff in yarn.lock
doesnt have the latest lock file @Airbornharsh
Updated
dont know why it's showing a whitespace diff in yarn.lock
I tried to remove it but showing whitespace
@hkirat check it out Solved lot of conflicts which took more time than creating these features
One big bug that should be fixed Time should not be client side but calculated on the server
PLease tag me once fixed
Check it out