Repository navigation
project finished - #287
project finished#287mikhailkrovikov wants to merge 2 commits into
Conversation
🚀 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. |
TheCSharpAcademy
left a comment
There was a problem hiding this comment.
@mikhailkrovikov Thanks for submitting. Before we can approve, could you please fix this bug? It looks like updating a record updates every record at the moment, could you please check? 😊
💁♂️If you have any questions, the best way to get help is in our Discord, in the #get-help-with-projects channel.
|
Oups, sorry! Fixed it |
|
@mikhailkrovikov Project approved! 😄✅ The project is clean and tidy, very easy to use and Spectre makes the UX great. Oh and you completed a bunch of challenges! Well done! 🥋You still need to submit your Freecodecamp certificate before you start earning belts 😊 Feedback 🔍️Validation shouldn't be handled by exceptions. Invalid input currently throws an exception, which is caught by the broad catch (Exception ex) in Program.Main, displays the error, and returns the user to the main menu. A validation loop that asks again would be friendlier. 🔍DeleteSessionCommand calculates deletedCount but never uses it, so it would be nice to report something like "3 sessions successfully deleted." 🔍Duration is stored as a string and sorted as a string. With the current fixed HH:mm:ss representation this works for normal same-day durations, but representing duration as a numeric value or TimeSpan would make calculations and future reporting/filtering more robust. ☕If you like our roadmap, please consider buying us a coffee. We appreciate your help 🙂 🗺️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 Overall great work!✋🏻Looking forward to seeing your next projects! |
please review my code tracker, thank you