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

Elizabeth Tan FTBC6 #58

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

Conversation

liztanyl
Copy link

Please fill out the survey before submitting the pull request. Thanks!

🚀🚀🚀🚀🚀🚀🚀🚀🚀🚀🚀🚀🚀🚀🚀🚀

How many hours did you spend on this assignment?
4-5hrs I think

Please fill in one error and/or error message you received while working on this assignment.
When I was setting a timeout for the cards to show and then be hidden again, I used setInterval() instead of setTimeout() and I was so confused as to why all my firstCards kept being 'turned over' 😂 finally figured it out when I explained the code, line by line, out loud to myself 🦆

What part of the assignment did you spend the most time on?
Understanding the game logic at the start! Subsequently it was adding the timer I think

Comfort Level (1-5): 3.5

Completeness Level (1-5): 5 (up to More comfortable - Match Game Timer)

What did you think of this deliverable? Kinda challenging but felt like good practice!

Is there anything in this code that you feel pleased about? I like how my game looks hehe 🥳

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.

1 participant