Skip to content

Coding Tracker with calendar display - #291

Open
Boudiz wants to merge 17 commits into
the-csharp-academy:mainfrom
Boudiz:main
Open

Boudiz wants to merge 17 commits into
the-csharp-academy:mainfrom
Boudiz:main

Conversation

@Boudiz

@Boudiz Boudiz commented Oct 12, 2026

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.

@Boudiz Thanks for submitting! The project needs a bit of work before approval:
🔍Improve database connection management; You're keeping a single SqliteConnection open throughout the application's lifetime. For a small console application, this might work, but it's best practice better to create and dispose of connections for individual database operations.

🔍There are no update and delete functions. If the user accidentally inputs a shift incorrectly it can't be undone.

🔍Your Readme doesn't contain enough information. We need at least a quick explanation of the design of the system and a reflection about your experience building the app.

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

@TheCSharpAcademy TheCSharpAcademy self-assigned this Oct 14, 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