-
Notifications
You must be signed in to change notification settings - Fork 145
Sudoko Puzzle #951
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
base: main
Are you sure you want to change the base?
Sudoko Puzzle #951
Conversation
✅ Deploy Preview for webdevin100days ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
Thanks for creating a PR for your Issue!
|
|
"Hey, your project name and demo link are not appearing in the table. You didn’t check the code or read the contributors.md and README.md files. You need to add your project name and demo link in app.js inside the script.js file. @Renu-code123 |
|
Hi @ruchikakengal , The main problem I’m facing is that my PRs are not getting merged because of repeated conflicts in app.js. Since other contributors are also updating the same file, their PRs often get merged first, and mine ends up in conflict every time. Because of this, many of my PRs have been left unmerged for a long time, which is becoming quite frustrating. Could you please guide me on the best way to handle this? If needed, I can create two separate PRs—one for my project and another for updating app.js—or, if possible, it would really help if my project PR could be merged first in such cases. I am not asking for points for the app.js update, only for the project PR I am adding. If you really want that, I must add my project to app.js. I will do it—I just hope this time I don’t get conflicts, because every time I have to delete my PR to resolve them. I don’t know if others are facing the same issue, but I wanted to share my concern so I can continue contributing effectively. I can only communicate with you about my concerns to get some help because I really want to contribute.
|
|
Okay, but in another PR, the app.js file should be updated. |
|
@ruchikakengal Okay, I will update the app.js in another PR, so please merge this PR |
|
@Renu-code123
Where are the remaining projects in your work?
Fix this issue; after that, I will merge this PR. |
|
Will you make another PR that includes all the project links? |
|
@ruchikakengal, I’m a bit confused 😭. If I add all my projects in one PR with updated app.js, how will points be assigned for each project separately? My idea is to make separate PRs for each project without updating app.js, so they can be merged without conflicts, and I get points for each project. After that, I’ll definitely create one final PR updating app.js, because without linking each project’s index.html in the repo, the deploy preview won’t show in the table. So please merge my project PRs first. If it’s acceptable under the project admin rules. Would this be okay for assigning points? I just want to follow the correct process without conflicts in my PRs and as per your rules. Whatever you suggest, I will follow. |




Description:
A fully functional Sudoku puzzle game built using HTML, CSS, and JavaScript.
Users can play a 9x9 Sudoku grid with real-time validation, difficulty levels, and optional hints, all within a clean and responsive UI.
✨ Features Implemented:
🎮 Interactive Sudoku Board – Users can click and enter numbers in each cell.
🏆 Difficulty Levels – Easy, Medium, Hard puzzle generator.
✅ Input Validation – Highlights incorrect moves (row/column/box).
💡 Hint System – Option to get hints for selected cells.
⏱️ Timer & Restart – Track completion time with reset/replay option.
📱 Responsive UI – Works seamlessly on both desktop and mobile.
🛠️ Tech Stack
✅ Type of Change
New Feature ✨
Fixes: #372
Demo :
Screen.Recording.2025-08-24.131125.mp4
@ruchikakengal Please review and merge this PR. I will add this project in app.js separately along with the others in a different PR to avoid conflicts.