Skip to content

Mathgame - #787

Closed
whoami3222 wants to merge 2 commits into
the-csharp-academy:mainfrom
whoami3222:main
Closed

whoami3222 wants to merge 2 commits into
the-csharp-academy:mainfrom
whoami3222:main

Conversation

@whoami3222

Copy link
Copy Markdown

No description provided.

@github-actions

Copy link
Copy Markdown

🚀 Project Submitted!

Thanks for submitting your project! I will review it soon.

📋 Please check the dashboard to make sure the project is marked as Pending.

⚠️ If it's not marked as pending, submit it manually using the URL for this PR.

@TheCSharpAcademy TheCSharpAcademy left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

@whoami3222 Thanks for submitting! The project needs a bit of work as it doesn't meet a couple of core requirements.

Image Image

There's no menu to choose operations and you're showing divisions that don't result in integers. Could you please fix? Once fix just commit the changes. No need for a new PR 😊

💁‍♂️If you have any questions, the best way to get help is in our Discord, in the #get-help-with-projects channel.

@TheCSharpAcademy

TheCSharpAcademy commented Oct 22, 2026 •

Copy link
Copy Markdown
Collaborator

@whoami3222 Project approved! 😄✅ And well done for completing the Random Operation challenge! I recommend you always do that, as it pushes you and makes you learn faster. I suggest you complete the other ones as well when you have the time, as it pushes you to learn more.

🥋And congratulations! You got your Green Belt!! 🟩🟩

Feedback
🔍️ The problem is tidy and well organized and you're using only one method for all operations. Great! 😍

🔍️In the menu, Invalid choice crashes the app:

image

🔍In the questions it would be better to give the users a message when they don't use integers and let them try again. image

👪I couldn't find your username (whoami222) in our Discord Community. Consider joining so you can be part of the community as well. 😁

☕If you like our roadmap, please consider buying as a coffee. We appreciate your help 🙂

Overall good work!✋🏻Looking forward to seeing your next projects!

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.

2 participants