Skip to content

Coding Tracker Project - Complete - #293

Closed
AaronjMurphy wants to merge 1 commit into
the-csharp-academy:mainfrom
AaronjMurphy:main
Closed

AaronjMurphy wants to merge 1 commit into
the-csharp-academy:mainfrom
AaronjMurphy:main

Conversation

@AaronjMurphy

Copy link
Copy Markdown

This project is a Coding Tracker based on requirements from The C# Academy.

@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 24, 2026 •

Copy link
Copy Markdown
Collaborator

@AaronjMurphy Project approved! Great work on your Coding Tracker! The app is very easy to use and ticks all boxes. Almost perfect except for a few details you'll see in the feedback below.👩‍🎤

🥋And congratulations! You got your Olive Green belt!! 🫒🫒

Feedback
🔍Several repository methods contain:

catch (Exception)
{
    throw;
}

These blocks simply rethrow exceptions without doing anything useful. You can remove them or introduce meaningful error handling so database failures don't unexpectedly crash the application.

🔍You've stored StartTime, EndTime, and Duration as strings. This works with SQLite, but using DateTime and TimeSpan in your C# model would make calculations and validation easier. You could then handle string conversion when saving and retrieving data from SQLite.

🔍Unit tests would have been great for this project, I strongly recommend you add it from the next project onwards.

☕If you like our roadmap, please consider buying us 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