Skip to content

Version 1.0 #1

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 12 commits into
base: master
Choose a base branch
from
Open

Version 1.0 #1

wants to merge 12 commits into from

Conversation

@raszta
Copy link

@raszta raszta commented Sep 1, 2019

Przesyłam do code review

damian0o
damian0o reviewed Sep 4, 2019
View reviewed changes
Copy link
Collaborator

@damian0o damian0o left a comment •
edited
Loading

Choose a reason for hiding this comment

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

General note -

  1. It is hard to tall which user is having his turn. Maybe some legend component with some informations will solved this issue.
  2. The one game is not entire match. Maybe to win you should win several games.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@damian0o damian0o damian0o left review comments

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@raszta @damian0o