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.

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

Allison ttt #30

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

Allison ttt #30

wants to merge 39 commits into from

Conversation

allisonhoke
Copy link

All waves complete.

guineveresaenger and others added 30 commits December 13, 2016 12:28
@guineveresaenger
created board, game, player files and specs, with proper imports and …
f9ec5c3 
…exports. Created basic Board attributes using BDD.
@allisonhoke
created attributes for player and created functions to set the attrib…
4f8e766 
…utes
@allisonhoke
created initial attributes of turnCounter, 2 players, and current pla…
6e52d80 
…yer for game
@guineveresaenger
Added checkRows to Board to check if the rows have a winning condition
1723fde
@guineveresaenger
Added win checks for diagonals, checkDiags
e6197ff
@allisonhoke
wrote function for checking the columns for a win
eac0b06
@allisonhoke
fixed merge conflicts
1818341
@allisonhoke
fixed more merge conflicts
4327001
@allisonhoke
created checkWin function to check all possibilities for winning game
52cea58
@allisonhoke
so many tests. created functions to check if a particular space is av…
ef5d979 
…ailable and to set a space with the given marker
@allisonhoke
adjusted setMarker function - we will handle the logic in the game. a…
497ac23 
…djusted test for function
@allisonhoke
added togglePlayer function with tests
402489e
@guineveresaenger
some setter methods for Game, and better logic in checkColumns
0744dfb
@guineveresaenger
fixed the flexibility for checking winning conditions on any size board
93aa0d1
@guineveresaenger
wrote comprehensive overall test for 1 gameplay with a win
cc2d870
@guineveresaenger
changed name of setMarker to makeMove for clarity. Also made all the …
6b92ea6 
…null board fields be a blank space string instead
@allisonhoke
set up file structure for backbone
ffc52b7
@allisonhoke
set up HTML structure
331f3d0
@guineveresaenger
set up import/export relations in file structure, declared most model…
c4036d1 
…s and views
@allisonhoke
added header to HTML
8af5acf
@guineveresaenger
Moved app.js one level up into src
6f2e1a2
@guineveresaenger
Added Css file and Foundation. Tweaked index to make a game board, pl…
b998ec3 
…ayers and new game button.
@allisonhoke
WIP working on DOM events
612439b
@allisonhoke
WIP working on DOM events, debugging lines in place
ffb4121
@guineveresaenger
resolved DOM event issues
313a9bb
@allisonhoke
turned playerr.js into backbone model and adjusted tests
ac8f75e
@allisonhoke
changed board.js into backbone model and adjusted tests to pass
872aa36
@allisonhoke
converted game.js into backbone model and adjusted tests
bc9dc91
@allisonhoke
adjusted to test that players have the correct names
e5c3fa8
@allisonhoke
adjusted defaults/initialize
fe1e19d
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
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
@allisonhoke @guineveresaenger