Skip to content

Conversation

@gloriafolaron
Copy link
Contributor

Summary

  • Re-enables the stopwatch partial in header menu (was commented out, causing htmx:targetError)
  • Fixes variable name passed to stopwatch partial (progressStepsonTheClock)
  • Adds error handling for punchIn failures (catches QueryException, validates session)
  • Shows success/error notifications when starting timer
  • Fixes template syntax (false|nullfalse)

Root Cause

The stopwatch partial in headMenu.blade.php was commented out, so the #timerHeadMenu element didn't exist. All timer buttons use hx-target="#timerHeadMenu", causing HTMX to fail silently.

Test plan

  • Start timer from My Work dashboard
  • Start timer from project kanban view (three-dot menu)
  • Start timer from company timesheets
  • Verify timer appears in header
  • Stop timer and verify time logged

Fixes #3208

🤖 Generated with Claude Code

- Re-enable stopwatch partial in header menu (was commented out)
- Fix variable name: progressSteps → onTheClock
- Add error handling for punchIn failures (QueryException, missing session)
- Add user feedback: success/error notifications when starting timer
- Fix template syntax: false|null → false

Fixes #3208

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@gloriafolaron gloriafolaron requested a review from a team as a code owner January 24, 2026 20:04
@CLAassistant
Copy link

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

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.

Unable to start the timer of a task

3 participants