Skip to content

Citlalli & Ana - CEDAR C16 #57

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 25 commits into
base: main
Choose a base branch
from
Open

Citlalli & Ana - CEDAR C16 #57

wants to merge 25 commits into from

Conversation

Anagabsoares
Copy link

No description provided.

Anagabsoares and others added 17 commits December 15, 2021 22:57
@Anagabsoares
Implement Square component
aebed3a
@Anagabsoares
CEdit variable declarations"
0390867 
"
@Anagabsoares
Delete console.log()
ec10840
@Anagabsoares
Refactor changeSquareValue function
d1deff2
@Anagabsoares
Added displayPlayer functionality
fd61c1f
@Anagabsoares
create pseudocode for check for winner
4544655
@MiffyBruna
generateSquareComponents now updated with maps also x o not hardcoded
58d4e23
@MiffyBruna
Finding winner algorithm solve, checks matches by column, rows and di…
e6f051a 
…agonals
@Anagabsoares
edit package JSON
dfca914
@Anagabsoares
Merge branch 'main' of https://github.com/Anagabsoares/react-tic-tac-toe
50e1205
@Anagabsoares
Implemented resetGame() function
91043fc
@MiffyBruna
Added logic to Reset Button
3aed5e5
@MiffyBruna
Reset Game works, merged changes
ae0475e
@Anagabsoares
Edit check for winner
170d08b
@Anagabsoares
"Edit app.js - not major changes"
6ba5f50
@MiffyBruna
Solution with refactoring, deleted flattenArray and changed how Board…
2555c8a 
… and Squares are rendered
@MiffyBruna
Merge pull request #1 from Anagabsoares/cz-branching
b5cc9e6 
Solution with refactoring, deleted flattenArray and changed how Board renders
@Anagabsoares Anagabsoares changed the title Citlali & Ana - CEDAR C16 Citlalli & Ana - CEDAR C16 Dec 21, 2021
kaidamasaki
kaidamasaki reviewed Jan 6, 2022
View reviewed changes
Copy link

@kaidamasaki kaidamasaki left a comment

Choose a reason for hiding this comment

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

Great job!

Everything looks really good! I don't have much to comment on. Well done!

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

Reviewers

1 more reviewer

@kaidamasaki kaidamasaki kaidamasaki left review comments

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@Anagabsoares @kaidamasaki @MiffyBruna