Skip to content

Single commit because stupid me did this in another repo - #790

Closed
vitorvelozo wants to merge 4 commits into
the-csharp-academy:mainfrom
vitorvelozo:main
Closed

vitorvelozo wants to merge 4 commits into
the-csharp-academy:mainfrom
vitorvelozo:main

Conversation

@vitorvelozo

Copy link
Copy Markdown

Missing tests for now. Did not have the time.
Tried for a MVC project structure but I don't know if it was the right choice.

@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 commented Oct 27, 2026 •

Copy link
Copy Markdown
Collaborator

@vitorvelozo Project approved! 😄✅ And well done for completing the challenges! I recommend you always do that, as it pushes you and makes you learn faster. Your project is clean, tidy, concise and to the point. And you're using only one method for all operations, great work!

🥋And congratulations! You got your green belt!! 🟩🟩

Feedback
🔍️ I can't say much about the code and the UI/UX. The project is close to impeccable. The only thing I'd do differently is show the actual questions in the history to make it more meaningful. Oh, and of course, as you said it yourself, unit tests. 😁

👪Oh and if possible join our Discord Community and add your username to your profile so you can be part of the community as well. 😁 I couldn't find a "stardy" in our Server.

Overall great 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