-
Notifications
You must be signed in to change notification settings - Fork 213
jzhartman CodingTracker Submission #238
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Conversation
Attempte at following MVC patter with Repositories and Connection Factories
…them rely on services
Got GetByDateRange working
Added Update StartTime feature
Working on overall app flow -- should write it up before continuing Pushing to remote to enable mobile access to structure for writeup on the go
Re-pushing to remote
Next step is adding session
Next - Manage the update/delete by Id
Refactored some items Working on update entry method
Only need to change confirmation message for no changes
Fixed usings and namespace indent in all files
Add session, stopwatch, Update, Delete, and Reports complete Needs Goals created Views need heavily refactored SessionsList printing needs serious help
About to jack up the views...
Changed printing to all use Spectre.Console Ugly, but funcitonal
…t happy with it, but done fiddling. Also fixed two minor UI bugs
…sion. Will need to apply it to Update as well. Also applied previous UI change for Delete Confirmation message to Add confirmation message.
Confirmed all validation cases for Add Session. Confirmed validation cases for Update Start Time Found error in validation for Update End Time Minor updates to UI
Next: Check validation for delete
…ses for CodingSessions and Goals Reworked DB initi class to accomodate new Goals table
Added beginning stages for goals implementation
Fixed date input
GetGoalValueFromUser built but needs tested Possibly change goalValue to TimeSpan
Next: Evaluate Goals
Next is update goal endt time
Print Completed Goal List done
Goals almost done Needs ExtendGoal method and then proofing
Bug fixes and testing complete Added goal extension to future items
Doublechecked namespace indentation
Added README.md with project overview, requirements, technologies, operation details, challenges, and future enhancements.
|
@jzhartman Project approved! ✅ And well done for completing the challenges! You left no stone unturned 😃. Except for the AI challenge, but I think only one person has completed that one so far so I won't give you a hard time 🤣 🥋And congratulations! You've got your Olive Green belt!! 🫒🫒 🔍️I'lI be honest, I don't have a lot of feedback, your project is pristine and I like how you're experimenting with design patterns. You should always push yourself into areas you feel you lack knowledge, regardless of our requirements/challenges.🤓. You probably noticed your services became a bit redundant and a bit of an overkill for a small project, but in real life they would probably do more. 💡What I suggest from now on is that you start adding unit testing to your projects. I'm making a push for this in the academy, having just created an article and a Youtube tutorial for this and I'm slowly adding it to the curriculum, but as I progress in my career I've been noticing it's one of the most important skills to have. ✋🏻Looking forward to seeing your next projects! |
No description provided.