diff --git a/containers/backupchecks/src/backend/app/main/routes_jobs.py b/containers/backupchecks/src/backend/app/main/routes_jobs.py index 5bf5ebe..ef4e93b 100644 --- a/containers/backupchecks/src/backend/app/main/routes_jobs.py +++ b/containers/backupchecks/src/backend/app/main/routes_jobs.py @@ -198,14 +198,10 @@ def job_detail(job_id: int): JOIN ticket_scopes ts ON ts.ticket_id = t.id WHERE ts.job_id = :job_id AND t.active_from_date <= :max_date - AND ( - COALESCE(ts.resolved_at, t.resolved_at) IS NULL - OR ((COALESCE(ts.resolved_at, t.resolved_at) AT TIME ZONE 'UTC' AT TIME ZONE :ui_tz)::date) >= :min_date - ) + AND COALESCE(ts.resolved_at, t.resolved_at) IS NULL """ ), - {"job_id": job.id, "min_date": min_date, - "ui_tz": _get_ui_timezone_name(), "max_date": max_date}, + {"job_id": job.id, "max_date": max_date}, ) .mappings() .all() @@ -240,14 +236,10 @@ def job_detail(job_id: int): r.active_from_date, ((r.start_date AT TIME ZONE 'UTC' AT TIME ZONE :ui_tz)::date) ) <= :max_date - AND ( - r.resolved_at IS NULL - OR ((r.resolved_at AT TIME ZONE 'UTC' AT TIME ZONE :ui_tz)::date) >= :min_date - ) + AND r.resolved_at IS NULL """ ), - {"job_id": job.id, "min_date": min_date, - "ui_tz": _get_ui_timezone_name(), "max_date": max_date}, + {"job_id": job.id, "max_date": max_date}, ) .mappings() .all() diff --git a/docs/changelog-claude.md b/docs/changelog-claude.md index 89f61cd..90a477b 100644 --- a/docs/changelog-claude.md +++ b/docs/changelog-claude.md @@ -7,6 +7,7 @@ This file documents all changes made to this project via Claude Code. ### Fixed - Fixed Autotask ticket not being automatically linked to new runs when internal ticket is resolved by implementing independent Autotask propagation strategy (now checks for most recent non-deleted and non-resolved Autotask ticket on job regardless of internal ticket status, ensuring PSA ticket reference persists across runs until explicitly resolved or deleted) - Fixed internal and Autotask tickets being linked to new runs even after being resolved by removing date-based "open" logic from ticket query (tickets now only link to new runs if they are genuinely unresolved, not based on run date comparisons) +- Fixed Job Details page showing resolved tickets in the Tickets column by removing date-based logic from display query (tickets and remarks now only show if genuinely unresolved, matching the linking behavior) ### Changed - Added debug logging to ticket linking function to troubleshoot resolved ticket propagation issues (writes to AuditLog table with event_type "ticket_link_debug", visible on Logging page, logs EVERY run import to show whether tickets were found and their resolved_at status, uses commit instead of flush to ensure persistence)