Skip to content

Done - #284

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

Done#284
Tenebris-06 wants to merge 4 commits into
the-csharp-academy:mainfrom
Tenebris-06:main

Conversation

@Tenebris-06

Copy link
Copy Markdown

No description provided.

@github-actions

github-actions Bot commented Sep 7, 2026

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

Copy link
Copy Markdown
Collaborator

@Tenebris-06 Thanks for submitting! I can't see this project available for review in the dashboard. Could you please submit in the dashboard before we can approve 😁.

@TheCSharpAcademy TheCSharpAcademy self-assigned this Sep 14, 2026
@Tenebris-06

Copy link
Copy Markdown
Author

Done!

@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.

@Tenebris-06 Thanks for the submission. For every project starting with 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).

Please add that before we can review. After making your changes, just commit and push, no need for a new pull request 😊.

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

@Tenebris-06

Copy link
Copy Markdown
Author

Sorry I completely forgot about that since I pushed this before the Habit Tracker.
Done as well

@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.

@Tenebris-06 Thanks for adding the Readme! Could you please fix this before we can approve?

⚠️ Date/time validation needs one correction: the requirements specifically state that the end date can't be before the start date. Currently the application simply calculates endTime - startTime, meaning something like 10:00 → 09:00 can be saved with a negative duration.

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

@Tenebris-06

Copy link
Copy Markdown
Author

I fixed it
Thanks for pointing that out

@TheCSharpAcademy

Copy link
Copy Markdown
Collaborator

@Tenebris-06 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!

🥋You're only one project away from getting your Olive Green belt!! 🫒🫒 The Habit Tracker 👏

Feedback
🔍️ The required functionality is well covered. You use Spectre.Console for menus, prompts, tables, confirmations, and the live-session display. The project is split into separate classes/files, and appsettings.json contains the SQLite connection string as required.

🔍One thing worth revisiting is Update. DataAccess.UpdateSession() exists, but there doesn't appear to be an update option exposed through MainMenu. Since this project inherits the CRUD expectations of the previous project, the user should be able to update an existing coding session from the UI as well. This is the main refinement I'd make to the submission.

🕵️‍♀️Also, have a look at our style guide since there are a few issues in your code. e.g. lots of unused usings, as per picture below. You can read our code conventions here🤓.

image

🗺️I noticed that you haven't completed any of our courses yet. I suggest you try our Intro to Object Oriented Programming course next. This will close gaps in your knowledge and give you more confidence about OOP

👪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. 😁

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

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

@Tenebris-06

Copy link
Copy Markdown
Author

Thanks a lot! will check things you noted for sure

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.

3 participants