Skip to content

Project Finished - #24

Open
LewisHardcastle wants to merge 2 commits into
the-csharp-academy:masterfrom
LewisHardcastle:master
Open

Project Finished#24
LewisHardcastle wants to merge 2 commits into
the-csharp-academy:masterfrom
LewisHardcastle:master

Conversation

@LewisHardcastle

Copy link
Copy Markdown

No description provided.

@TheCSharpAcademy TheCSharpAcademy left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

@LewisHardcastle Thanks for the submission. For every project after the Habit Tracker one of the requirements is a Readme with:
✅ Instructions on how to run the project
✅ Explain what the app does and how it works
✅ Explain architectural choices
✅ Reflection on your experience while developing it (ideally written with your own words)

Also:
⚠️ Update/Delete nonexistent records This needs fixing before approval. There's actually a comment in the code reminding you:

// Laeve a message for when user inputs a habit that does not exist

⚠️ Menu validation. The default branch of your menu currently does nothing:

default:
    //
    break;

An invalid option should tell the user that their selection isn't valid rather than silently continuing.

Please fix these before we can review.

@TheCSharpAcademy TheCSharpAcademy self-assigned this Aug 22, 2026
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