feat: Add setting to enforce comments on timesheets

[akash2017sky]

Summary

• Adds a "Require Comments on Time Entries" toggle in Settings that enforces notes/comments when logging time
• When enabled, the TimeEntryModal validates that notes are filled before saving and labels the field as "required"
• Timesheet submission is blocked if any entries are missing comments, with a clear error message

Screenshots

Closes #66

Test plan

• Toggle "Require Comments on Time Entries" ON in Settings
• Try saving a time entry without notes — should show an error toast
• Try submitting a timesheet with entries missing notes — should show an error with count
• Toggle the setting OFF — notes should revert to optional behavior
• Verify the setting persists across page reloads

[Copilot]

Pull request overview

Adds a new user setting to require notes/comments on time entries, and enforces it in both the time entry modal and timesheet submission flow.

Changes:

• Adds requireTimesheetComments to UserSettings and initializes it in the settings store defaults.
• Adds a Settings toggle to enable/disable requiring comments on time entries.
• Enforces the setting by validating notes on save (modal) and blocking timesheet submission when any entry is missing notes.

Reviewed changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated 2 comments.

Show a summary per file

File	Description
src/types/index.ts	Extends UserSettings with requireTimesheetComments.
src/hooks/useSettingsStore.ts	Adds default value for the new setting in the persisted settings store.
src/components/timesheet/WeeklyTimesheet.tsx	Blocks submission when the setting is enabled and entries are missing notes.
src/components/timesheet/TimeEntryModal.tsx	Validates notes presence when the setting is enabled and updates UI labeling/required flag.
src/components/settings/SettingsPanel.tsx	Adds a UI toggle wired to useSettingsStore to control the setting.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[EdiWeeks]

@akash2017sky please take a look above!

[EdiWeeks]

Tested, Copilot review comments addressed. @BenGWeeks please merge

[BenGWeeks]

Architecturally this is storing in UserSettings, what we want is a company wide setting, so we need to consider where that should be set and who can update it.