Skip to content
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

Dianne Tic Tac Toe #24

Open
wants to merge 69 commits into
base: master
Choose a base branch
from
Open

Dianne Tic Tac Toe #24

wants to merge 69 commits into from

Conversation

dlaguerta
Copy link

@dlaguerta dlaguerta commented Dec 23, 2016

Finished Waves 1, 2, and 3, but not Wave 4.
The game has a good flow, with a few bugs that I found at the end of the day Friday. Bugs include:

  • Not showing the "start game over" or "space is taken" messages to players after a new game starts (everything works fine in the first game, but these particular elements never get reset again?)

  • Appending messages together without clearing out previous messages (ex. when you try to place a marker in an occupied space and have already clicked that same space.). It works when you do not click on the same space after you receive the warning.

Overall, very fun! I will stick to playing tic tac toe on paper though.
P.S. I loved working with Danielle!

dschrimm and others added 30 commits December 13, 2016 11:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants