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

Feat/memory challenge #86

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

Conversation

kimcascante
Copy link

@kimcascante kimcascante commented Feb 1, 2025

📝 Summary
This PR enhances the Memory Challenge game by starting with a smaller grid and increasing the grid size and sequence length as the user progresses.

Related Issues
Closes #61

Type of Change
📝 [ ] Documentation (updates to README, docs, or comments)
🐛 [ ] Bug fix (non-breaking change which fixes an issue)
👌 [x] Enhancement (non-breaking change which adds functionality)
💥 [ ] Breaking change (fix or feature that would cause existing functionality to change)

🔄 Changes Made

  • The game now starts with a 2x2 grid and increases up to 6x6 as the user successfully completes levels.

Implementation Details
Adjusted the sequence generation logic to match the current grid size.
Conditional rendering of the "Restart Game" button based on the user's success or failure.

Technical Notes
The grid size and sequence length are managed using React state hooks.

🔧 Tests Results

[ ] Unit Tests
[ ] Integration Tests
[x] Manual Testing

Evidence
Screenshot 2025-01-31 at 5 14 12 PM

screen-recording-2025-01-30-at-60817-am_fKrcWOxS.mp4

Testing Notes
Verified that the grid size increases correctly after each successful level.

🔜 Next Steps
Explore potential UI/UX improvements based on user feedback.

Copy link

vercel bot commented Feb 1, 2025

@kimcascante is attempting to deploy a commit to the ByteBuilders Team on Vercel.

A member of the Team first needs to authorize it.

@kimcascante
Copy link
Author

@juandiegocv27 Hi, could you please review and share feedback?

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.

Minigame: Memory Challenge
1 participant