firefighter-incident 0.0.35__py3-none-any.whl → 0.0.37__py3-none-any.whl

firefighter/raid/signals/incident_updated.py

@@ -3,17 +3,46 @@ from __future__ import annotations
 import logging
 from typing import TYPE_CHECKING, Any
 
+from django.conf import settings
+from django.core.cache import cache
+from django.db.models.signals import post_save
 from django.dispatch.dispatcher import receiver
 
 from firefighter.incidents.enums import IncidentStatus
+from firefighter.incidents.models.incident import Incident
 from firefighter.incidents.signals import incident_updated
-from firefighter.raid.client import client
+from firefighter.raid.client import RAID_JIRA_WORKFLOW_NAME, client
+from firefighter.raid.utils import normalize_cache_value
 
 if TYPE_CHECKING:
-    from firefighter.incidents.models.incident import Incident
     from firefighter.incidents.models.incident_update import IncidentUpdate
 
 logger = logging.getLogger(__name__)
+JIRA_SYNC_CACHE_TIMEOUT = getattr(settings, "JIRA_SYNC_CACHE_TIMEOUT", 60)
+
+JIRA_STATUS_INCOMING = "Incoming"
+JIRA_STATUS_PENDING_RESOLUTION = "Pending resolution"
+JIRA_STATUS_IN_PROGRESS = "in progress"
+JIRA_STATUS_REPORTER_VALIDATION = "Reporter validation"
+JIRA_STATUS_CLOSED = "Closed"
+
+IMPACT_TO_JIRA_STATUS_MAP: dict[IncidentStatus, str] = {
+    IncidentStatus.OPEN: JIRA_STATUS_INCOMING,
+    IncidentStatus.INVESTIGATING: JIRA_STATUS_IN_PROGRESS,
+    IncidentStatus.MITIGATING: JIRA_STATUS_IN_PROGRESS,
+    IncidentStatus.MITIGATED: JIRA_STATUS_REPORTER_VALIDATION,
+    IncidentStatus.POST_MORTEM: JIRA_STATUS_REPORTER_VALIDATION,
+    IncidentStatus.CLOSED: JIRA_STATUS_CLOSED,
+}
+
+
+def _set_impact_to_jira_cache(
+    incident_id: Any, field: str, value: Any, timeout: int = JIRA_SYNC_CACHE_TIMEOUT
+) -> None:
+    cache_key = (
+        f"sync:impact_to_jira:{incident_id}:{field}:{normalize_cache_value(value)}"
+    )
+    cache.set(cache_key, value=True, timeout=timeout)
 
 
 @receiver(signal=incident_updated, sender="update_status")
@@ -31,29 +60,315 @@ def incident_updated_close_ticket_when_mitigated_or_postmortem(
     - P3+ (no postmortem): Close when incident is MITIGATED or CLOSED
     - POST_MORTEM status never closes the ticket (it remains open during PM phase)
     """
+    logger.debug(
+        "incident_updated handler invoked for incident #%s with status %s; updated_fields=%s event_type=%s",
+        getattr(incident, "id", "unknown"),
+        incident_update.status,
+        updated_fields,
+        incident_update.event_type,
+    )
+    # Skip if this update was produced by Jira webhook sync to avoid redundant close calls
+    if incident_update.event_type == "jira_status_sync":
+        logger.debug(
+            "Skipping Jira transition: incident #%s update came from Jira (event_type=jira_status_sync)",
+            getattr(incident, "id", "unknown"),
+        )
+        return
+
     if "_status" not in updated_fields:
+        logger.debug(
+            "Skipping Jira transition: incident #%s update lacks _status in updated_fields (%s)",
+            getattr(incident, "id", "unknown"),
+            updated_fields,
+        )
         return
 
     if not hasattr(incident, "jira_ticket") or incident.jira_ticket is None:
         logger.warning(
-            f"Trying to close Jira ticket for incident {incident.id} but no Jira ticket found"
+            "Trying to close Jira ticket for incident %s but no Jira ticket found",
+            getattr(incident, "id", "unknown"),
+        )
+        return
+
+    # Special case: when Impact moves to MITIGATING, Jira must go through two steps:
+    # "Pending resolution" then "in progress".
+    if incident_update.status == IncidentStatus.MITIGATING:
+        all_steps_succeeded = True
+        for step in (JIRA_STATUS_PENDING_RESOLUTION, JIRA_STATUS_IN_PROGRESS):
+            try:
+                logger.debug(
+                    "Transitioning Jira ticket %s via workflow %s to status %s (incident #%s, impact status %s)",
+                    incident.jira_ticket.id,
+                    RAID_JIRA_WORKFLOW_NAME,
+                    step,
+                    getattr(incident, "id", "unknown"),
+                    incident_update.status,
+                )
+                client.transition_issue_auto(
+                    incident.jira_ticket.id, step, RAID_JIRA_WORKFLOW_NAME
+                )
+            except Exception:
+                all_steps_succeeded = False
+                logger.exception(
+                    "Failed to transition Jira ticket %s to %s for incident %s",
+                    incident.jira_ticket.id,
+                    step,
+                    getattr(incident, "id", "unknown"),
+                )
+        if all_steps_succeeded:
+            logger.info(
+                "Transitioned Jira ticket %s through Pending resolution -> in progress from Impact status %s",
+                incident.jira_ticket.id,
+                incident_update.status.label if incident_update.status else "Unknown",
+            )
+        else:
+            logger.warning(
+                "At least one Jira transition failed while moving ticket %s to MITIGATING (incident #%s)",
+                incident.jira_ticket.id,
+                getattr(incident, "id", "unknown"),
+            )
+        return
+
+    # Decide target Jira status based on Impact status and postmortem requirement.
+    # P3+ (no postmortem): close Jira when Impact reaches MITIGATED or CLOSED.
+    # P1/P2 (needs_postmortem): close Jira only when Impact reaches CLOSED.
+    incident_status = incident_update.status
+    if incident_status is None:
+        logger.info(
+            "Skipping Jira transition: incident #%s status is None",
+            getattr(incident, "id", "unknown"),
+        )
+        return
+
+    target_jira_status: str | None = (
+        JIRA_STATUS_CLOSED
+        if incident_status == IncidentStatus.CLOSED
+        else IMPACT_TO_JIRA_STATUS_MAP.get(incident_status)
+    )
+
+    if target_jira_status is None:
+        logger.info(
+            "Skipping Jira transition: no Jira status mapping for Impact status %s (incident #%s)",
+            incident_update.status,
+            getattr(incident, "id", "unknown"),
+        )
+        return
+
+    try:
+        incident_id = getattr(incident, "id", None)
+        if incident_id is not None:
+            _set_impact_to_jira_cache(incident_id, "status", target_jira_status)
+        logger.debug(
+            "Transitioning Jira ticket %s via workflow %s to status %s (incident #%s, impact status %s)",
+            incident.jira_ticket.id,
+            RAID_JIRA_WORKFLOW_NAME,
+            target_jira_status,
+            getattr(incident, "id", "unknown"),
+            incident_update.status,
+        )
+        client.transition_issue_auto(
+            incident.jira_ticket.id, target_jira_status, RAID_JIRA_WORKFLOW_NAME
+        )
+        logger.info(
+            "Transitioned Jira ticket %s to %s from Impact status %s",
+            incident.jira_ticket.id,
+            target_jira_status,
+            incident_update.status.label if incident_update.status else "Unknown",
+        )
+    except Exception:
+        logger.exception(
+            "Failed to transition Jira ticket %s to %s for incident %s",
+            incident.jira_ticket.id,
+            target_jira_status,
+            getattr(incident, "id", "unknown"),
+        )
+
+
+# Listen to all incident_updated signals so both UI (update_status) and API/admin paths trigger
+@receiver(signal=incident_updated)
+def incident_updated_sync_priority_to_jira(
+    sender: Any,
+    incident: Incident,
+    incident_update: IncidentUpdate,
+    updated_fields: list[str],
+    **kwargs: Any,
+) -> None:
+    """Push Impact priority changes to Jira custom priority field (customfield_11064).
+    Skips if change originated from Jira (event_type='jira_priority_sync') to avoid loops.
+    """
+    logger.debug(
+        "Priority sync handler invoked: incident #%s sender=%s updated_fields=%s event_type=%s",
+        getattr(incident, "id", "unknown"),
+        sender,
+        updated_fields,
+        incident_update.event_type,
+    )
+
+    if incident_update.event_type == "jira_priority_sync":
+        logger.debug(
+            "Skipping Jira priority sync: incident #%s update came from Jira (event_type=jira_priority_sync)",
+            getattr(incident, "id", "unknown"),
+        )
+        return
+
+    if "priority_id" not in updated_fields:
+        logger.debug(
+            "Skipping Jira priority sync: incident #%s update lacks priority_id in updated_fields (%s) sender=%s",
+            getattr(incident, "id", "unknown"),
+            updated_fields,
+            sender,
+        )
+        return
+
+    if not hasattr(incident, "jira_ticket") or incident.jira_ticket is None:
+        logger.debug(
+            "Skipping Jira priority sync: incident #%s has no Jira ticket",
+            getattr(incident, "id", "unknown"),
+        )
+        return
+
+    if not incident.priority:
+        logger.debug(
+            "Skipping Jira priority sync: incident #%s priority is missing",
+            getattr(incident, "id", "unknown"),
         )
         return
 
-    # Determine if we should close the ticket based on status and priority
-    should_close = False
+    try:
+        incident_id = getattr(incident, "id", None)
+        if incident_id is not None:
+            _set_impact_to_jira_cache(incident_id, "priority", incident.priority.value)
+        client.update_issue_fields(
+            incident.jira_ticket.id,
+            customfield_11064={"value": str(incident.priority.value)},
+        )
+        logger.info(
+            "Synced priority %s to Jira ticket %s (customfield_11064) for incident #%s",
+            incident.priority.value,
+            incident.jira_ticket.id,
+            getattr(incident, "id", "unknown"),
+        )
+    except Exception:
+        logger.exception(
+            "Failed to sync priority %s to Jira ticket %s for incident %s",
+            incident.priority.value,
+            incident.jira_ticket.id,
+            getattr(incident, "id", "unknown"),
+        )
+
+
+# Fallback: if an Incident save bypasses incident_updated (e.g., admin inline), push priority anyway.
+@receiver(post_save, sender=Incident)
+def incident_priority_post_save_fallback(
+    sender: Any,
+    instance: Incident,
+    *,
+    created: bool,
+    update_fields: set[str] | None,
+    **kwargs: Any,
+) -> None:
+    """Fallback to push priority to Jira when Incident saves with priority_id in update_fields
+    but no incident_updated signal fired (e.g., admin edits). Skips when marked to avoid loops.
+    """
+    if created:
+        return
+    if update_fields and "priority_id" not in update_fields:
+        return
+    if getattr(instance, "_skip_priority_sync", False):
+        logger.debug(
+            "Skipping post_save priority sync for incident #%s due to skip flag",
+            getattr(instance, "id", "unknown"),
+        )
+        return
+    if not hasattr(instance, "jira_ticket") or instance.jira_ticket is None:
+        logger.debug(
+            "Skipping post_save priority sync: incident #%s has no Jira ticket",
+            getattr(instance, "id", "unknown"),
+        )
+        return
+    if not instance.priority:
+        logger.debug(
+            "Skipping post_save priority sync: incident #%s priority missing",
+            getattr(instance, "id", "unknown"),
+        )
+        return
 
-    if incident_update.status == IncidentStatus.CLOSED:
-        # Always close on CLOSED regardless of priority
-        should_close = True
-    elif incident_update.status == IncidentStatus.MITIGATED:
-        # Only close on MITIGATED if incident doesn't need postmortem (P3+)
-        should_close = not incident.needs_postmortem
+    try:
+        _set_impact_to_jira_cache(instance.id, "priority", instance.priority.value)
+        client.update_issue_fields(
+            instance.jira_ticket.id,
+            customfield_11064={"value": str(instance.priority.value)},
+        )
+        logger.info(
+            "Post-save synced priority %s to Jira ticket %s (customfield_11064) for incident #%s",
+            instance.priority.value,
+            instance.jira_ticket.id,
+            getattr(instance, "id", "unknown"),
+        )
+    except Exception:
+        logger.exception(
+            "Failed post-save priority sync %s to Jira ticket %s for incident %s",
+            instance.priority.value,
+            instance.jira_ticket.id,
+            getattr(instance, "id", "unknown"),
+        )
 
-    # POST_MORTEM status never closes the ticket - it stays open during PM phase
 
-    if should_close:
-        status_label = incident_update.status.label if incident_update.status else "Unknown"
-        logger.info(f"Closing Jira ticket for incident {incident.id} (status: {status_label})")
-        client.close_issue(issue_id=incident.jira_ticket.id)
-        # XXX We may want to add a comment if there is an incident update message on close
+@receiver(post_save, sender=Incident)
+def incident_status_post_save_fallback(
+    sender: Any,
+    instance: Incident,
+    *,
+    created: bool,
+    update_fields: set[str] | None,
+    **kwargs: Any,
+) -> None:
+    """Fallback to push status to Jira when Incident saves with status in update_fields
+    but no incident_updated signal fired (e.g., admin edits). Skips when marked to avoid loops.
+    """
+    if created:
+        return
+    if (
+        update_fields
+        and "_status" not in update_fields
+        and "status" not in update_fields
+    ):
+        return
+    if getattr(instance, "_skip_status_sync", False):
+        logger.debug(
+            "Skipping post_save status sync for incident #%s due to skip flag",
+            getattr(instance, "id", "unknown"),
+        )
+        return
+    if not hasattr(instance, "jira_ticket") or instance.jira_ticket is None:
+        logger.debug(
+            "Skipping post_save status sync: incident #%s has no Jira ticket",
+            getattr(instance, "id", "unknown"),
+        )
+        return
+    target_jira_status = IMPACT_TO_JIRA_STATUS_MAP.get(instance.status)
+    if target_jira_status is None:
+        logger.debug(
+            "Skipping post_save status sync: no Jira mapping for status %s (incident #%s)",
+            instance.status,
+            getattr(instance, "id", "unknown"),
+        )
+        return
+    try:
+        _set_impact_to_jira_cache(instance.id, "status", target_jira_status)
+        client.transition_issue_auto(
+            instance.jira_ticket.id, target_jira_status, RAID_JIRA_WORKFLOW_NAME
+        )
+        logger.info(
+            "Post-save synced status %s to Jira ticket %s for incident #%s",
+            instance.status,
+            instance.jira_ticket.id,
+            getattr(instance, "id", "unknown"),
+        )
+    except Exception:
+        logger.exception(
+            "Failed post-save status sync %s to Jira ticket %s for incident %s",
+            instance.status,
+            instance.jira_ticket.id,
+            getattr(instance, "id", "unknown"),
+        )

firefighter/raid/utils.py

@@ -2,9 +2,7 @@ from __future__ import annotations
 
 import logging
 from functools import cache
-from typing import TypeVar
-
-T = TypeVar("T")
+from typing import Any
 
 logger = logging.getLogger(__name__)
 
@@ -40,3 +38,15 @@ def get_domain_from_email(email: str) -> str:
     domain_parts = domain.split(".")
 
     return (".".join(domain_parts[-2:]) if len(domain_parts) > 2 else domain).lower()
+
+
+def normalize_cache_value(value: Any) -> str:
+    """Normalize cache values for loop-prevention keys."""
+    if value is None:
+        return ""
+    if isinstance(value, str):
+        return value.strip().lower()
+    try:
+        return str(int(value))
+    except (TypeError, ValueError):
+        return str(value).strip().lower()

{firefighter_incident-0.0.35.dist-info → firefighter_incident-0.0.37.dist-info}/METADATA

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: firefighter-incident
-Version: 0.0.35
+Version: 0.0.37
 Summary: Incident Management tool made for Slack using Django
 Project-URL: Repository, https://github.com/ManoManoTech/firefighter-incident
 Project-URL: Documentation, https://manomanotech.github.io/firefighter-incident/latest/

{firefighter_incident-0.0.35.dist-info → firefighter_incident-0.0.37.dist-info}/RECORD

@@ -6,7 +6,7 @@ gunicorn.conf.py,sha256=vHsTGjaKOr8FDMp6fTKYTX4AtokmPgYvvt5Mr0Q6APc,273
 main.py,sha256=Brj7IANCvq7zHGT7mm_VDO1_vV7OFwt6Zpt4gUwP4pM,1532
 manage.py,sha256=5ivHGD13C6nJ8QvltKsJ9T9akA5he8da70HLWaEP3k8,689
 firefighter/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
-firefighter/_version.py,sha256=w77E3DIE0tp22UaCyYn9461JfWMpEPhsv91t-4n4Bjw,706
+firefighter/_version.py,sha256=BV1Ma7idHU0hCvzfaqU-vGA4N-4x9mQtKafasKpzadA,706
 firefighter/api/__init__.py,sha256=JQW0Bv6xwGqy7ioxx3h6UGMzkkJ4DntDpbvV1Ncgi8k,136
 firefighter/api/admin.py,sha256=Q6f37xwf-i0xypFx6zU7r6bYxsSvLm66naZSHUK13JM,4621
 firefighter/api/apps.py,sha256=P5uU1_gMrDfzurdMbfqw1Bnb2uNKKcMq17WBPg2sLhc,204
@@ -186,13 +186,13 @@ firefighter/incidents/models/__init__.py,sha256=FLVyBwIdyxLdgSvXRAKC3fry9Ywwqlqh
 firefighter/incidents/models/environment.py,sha256=51txwua3dCrWZ1iSG3ZA8rbDn9c00pyMAZujl9gwE5c,827
 firefighter/incidents/models/group.py,sha256=VrVL315VFUvKW69AZuRUBg1h0jZJvn8zWeMxMOWec1Y,700
 firefighter/incidents/models/impact.py,sha256=D9NngMtg4XdDWnMgdVYaWCoUZ-fMXTvfL0eTEk9sc7M,4854
-firefighter/incidents/models/incident.py,sha256=YM-mQXoNcvO5BBdUjDqfvb6zPQmFO281qDa8PIQJ6Y4,29621
+firefighter/incidents/models/incident.py,sha256=qyg_1NB4cppkcPpwHeswynVVc2Dx2Ttd3fIUkXI9HTg,30337
 firefighter/incidents/models/incident_category.py,sha256=FSUkI9pOYddzVscGE0bpz_axWgBXDE8-5sf1oDZ_784,7714
 firefighter/incidents/models/incident_cost.py,sha256=juwOfJKRaNQpOHkRUCHShDDba0FU98YjRPkU4I0ofAU,1346
 firefighter/incidents/models/incident_cost_type.py,sha256=wm8diry_VySJzIjC9M3Yavv2tYbvJgpN9UDb2gFRuH4,845
 firefighter/incidents/models/incident_membership.py,sha256=vvvBvYPxNlM98KdF81cMrDif8_Wl5TqqNkmf_z9lZO8,1745
 firefighter/incidents/models/incident_role_type.py,sha256=YxtQmsmZQRVxa_pSVe-lIVEoZN5k56cR7o8soWCcFng,2104
-firefighter/incidents/models/incident_update.py,sha256=lCslA0Bz7qI9qORr0-Wir6yWzp0AeuCKsYHhLyKKukQ,4612
+firefighter/incidents/models/incident_update.py,sha256=TO8moYMHRGbmZmzs3yWiN1kwfF3i7vDOx7lXmW6iidc,4437
 firefighter/incidents/models/metric_type.py,sha256=nmc7LANTMjwQmm6LCfLhv4aF7ClEBV2L4PAu1wGfV3w,2981
 firefighter/incidents/models/milestone_type.py,sha256=SToXfQncNkGWLs4xZDNtT0lM4lj-9Pj9mZvr4JI85Fs,1827
 firefighter/incidents/models/priority.py,sha256=5bWsuurbE6TFtMczgCBX6uYkb-nBdTC2ABV-d7b9iw0,2199
@@ -200,7 +200,7 @@ firefighter/incidents/models/severity.py,sha256=a9p0l8oexr1Ve6pBIz-1rpUzDJAZ-PxH
 firefighter/incidents/models/user.py,sha256=Aok7g3d0uB7_gq8aXSYjEt7ogPpCkGX9JxNyjjE8XsU,3193
 firefighter/incidents/static/css/incident.css,sha256=48f03RWvchTz2Te2xSBTRd28qOlLO212sxbGWBd7d2M,2811
 firefighter/incidents/static/css/main.css,sha256=ZYaP3CbVQ1PMjmaWq91SQAbUBhW1BciuzJ52dvJI0fM,48
-firefighter/incidents/static/css/main.min.css,sha256=P9bd2u-TRtZClyutuYw7R5JT9YZtS-ogWiWhZVVAl0M,119104
+firefighter/incidents/static/css/main.min.css,sha256=6uPLzrqwP-Gkl_XjHt4xRuDIUQ_vTJ5GHmOAVVdPeUQ,120279
 firefighter/incidents/static/css/tailwind.css,sha256=afzjUc4YqJ6TYcYEWJbEMyx6fZ4m9b1ORqMpGTdfps8,977
 firefighter/incidents/static/img/gameday.png,sha256=CaoUFsi5GsdcVOfLFOx-8zpux-gwbj4a4V8bmfWk-x0,6391
 firefighter/incidents/static/img/logo-firefighter.png,sha256=eN7FJa9I1rj6-ohk4nEah8v0aBfHheeShjFWGMciRz0,2090
@@ -251,7 +251,7 @@ firefighter/incidents/templates/pages/docs_metrics.html,sha256=q10CCPwjujuj9_h7M
 firefighter/incidents/templates/pages/incident_category_detail.html,sha256=65vhQ3l3cWlLUYilfKHPuMn7lEfRJFjokiOeYUpeTTY,5425
 firefighter/incidents/templates/pages/incident_category_list.html,sha256=gXLPk9N7R585xqmA6JpL06J7hsxAXQvnsShesK2HR9o,1731
 firefighter/incidents/templates/pages/incident_create.html,sha256=syDS8EqmdcCf6z2dJox5gSPTOsyNdq0jFiDdzVc_vmU,2280
-firefighter/incidents/templates/pages/incident_detail.html,sha256=HhwpMg1nAaUqbYDi2RrgK5IkPhiKzHKcoSmk98O-rng,15058
+firefighter/incidents/templates/pages/incident_detail.html,sha256=7KbrQMymcPUw0qbF4m08V9afsxrWxml6iGzdk-q5qPk,16238
 firefighter/incidents/templates/pages/incident_list.html,sha256=IWTZyuIeG1RVCZGNLQ_N3Jg7a_Ush6YbQXWUy0xVuV8,2339
 firefighter/incidents/templates/pages/incident_role_types_detail.html,sha256=LVoFbW9rrP4Yln_Ld-Jwe1JvhhvgwpxZgpdNzbrLWsA,4099
 firefighter/incidents/templates/pages/incident_role_types_list.html,sha256=SRBhDIfne-bX4C-EhYqyCktIksR4Rd5GPMkYfE6El1o,1556
@@ -264,7 +264,7 @@ firefighter/incidents/views/date_filter.py,sha256=fUhTjkBulMokI5tAHuqNDVv1dyspjm
 firefighter/incidents/views/date_utils.py,sha256=Q2i-84hXm1vuXy7-1x1SSteWXzXUzh4b0a79nmTEfWA,5733
 firefighter/incidents/views/errors.py,sha256=yDuH0YOdGf-voVNEC51yR9Ie3OU-az7g2EqWs_uV1Kk,7855
 firefighter/incidents/views/reports.py,sha256=1Iegx04w-oHw4cj7u9w2_s7T_e9FH5I6RRPTwDZwZhg,20973
-firefighter/incidents/views/views.py,sha256=4AR3i9SiC9AnrzZCRIhx147rus4_dXS0IRWsQ8ilkjg,11031
+firefighter/incidents/views/views.py,sha256=ynMYwEi4vKHpYwrH6kEAiWKY91NVF8QSZfKy5thQuXg,12287
 firefighter/incidents/views/components/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
 firefighter/incidents/views/components/details.py,sha256=gFEezmL1TcVYnM_ryLNNMaynuIdjYV31Qzx_GfzrQiA,1040
 firefighter/incidents/views/components/list.py,sha256=u8HfXetmdL59h_4AZIhiHmKcmrPRZXgekPfnucB4Rek,2207
@@ -328,22 +328,22 @@ firefighter/pagerduty/views/oncall_trigger.py,sha256=LYHpWyEaR6O8NazmsTl5ydtw1XH
 firefighter/raid/__init__.py,sha256=nMNmvHCSkyLQsdhTow7myMU62vXk1e755gUntVfFFlY,154
 firefighter/raid/admin.py,sha256=WhIHaRAv7JPp2NH27w7_0JfvGHrvoyRJhYr3_WwedrA,1117
 firefighter/raid/apps.py,sha256=olDKua1rqhhIJUhCu6A2PnPWloW_jbeD4XWL94b2owo,1117
-firefighter/raid/client.py,sha256=txHIRPxvJkcg40XJGseyj3KOOVep8WMdrlDhJ5oVP_Q,8085
+firefighter/raid/client.py,sha256=03YJFZYkJe7PsgjLXAISOwETKd4KBkMJjyOOuAv5I08,8336
 firefighter/raid/forms.py,sha256=XivNADFHOl2ewNRXev17HthDIUvwt4pdH9t4So-BE-A,11968
 firefighter/raid/messages.py,sha256=e75kwi0hCe5ChwU4t-_6Q3Rcy22MLLdVSsYyjvG2SCM,5542
 firefighter/raid/models.py,sha256=29Smci739K1ZdcMu7uXYvoVEhgDpwLQoCzBbc5wvwhs,2211
 firefighter/raid/resources.py,sha256=39GhITs3OAWA1eSPZme-rLd818kuz7gwYzdN38zNz8Y,436
-firefighter/raid/serializers.py,sha256=4U6UFUNVR68lBYDzjYp8j1JzmfSftASOp1nQifCRXYQ,11941
+firefighter/raid/serializers.py,sha256=MGswzLV1o4Ke19szyfevWzuWeVVNW6LPopXbwhc4nHg,22327
 firefighter/raid/service.py,sha256=tkMttdao_4s7In66jQhN1CN0UFzR2OeXYSPQDXfLz0I,8577
 firefighter/raid/types.py,sha256=E0ZUjBCMwCgr0eSX3CfRTpmq0n-jMans1WxoYSnj7xg,477
 firefighter/raid/urls.py,sha256=oESkDY2tfZcnPGUgULqixvbV3Z7YsZfeI10RX3A5tZY,924
-firefighter/raid/utils.py,sha256=OhsuFb46lokVxjyfeBFLMcMIGzjkTrrdd6DCXUJym0E,1111
+firefighter/raid/utils.py,sha256=QRNhjZSfYzxjVLHgpoH3rneYaH53IBoE0cCEGYrGC6U,1422
 firefighter/raid/migrations/0001_initial_oss.py,sha256=oZQ44dHboLeHbvVyziHX_hhdis8H_Buv2W3jHiDzQhA,6259
 firefighter/raid/migrations/0002_featureteam_remove_qualifierrotation_jira_user_and_more.py,sha256=1UkwuNqqjtRJ_xlgNMs9qn2ryLcssQN06ZBCL296CWw,1034
 firefighter/raid/migrations/0003_delete_raidarea.py,sha256=M_XkKCu73ib2H09co2L-ssLQakJJFNOfqJpKrzOYP2Y,332
 firefighter/raid/migrations/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
 firefighter/raid/signals/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
-firefighter/raid/signals/incident_updated.py,sha256=Al1S1BOWAd6s97ITG0Yg181hOOLR5Byx8OO9hM5kP9E,2231
+firefighter/raid/signals/incident_updated.py,sha256=2rT7KLCyOZwd9HovmYs9_h61aKL-6gwKqBP-144-BsA,13957
 firefighter/raid/tasks/__init__.py,sha256=U4S_2y3zgLZVfMenHRaJFBW8yqh2mUBuI291LGQVOJ8,35
 firefighter/raid/views/__init__.py,sha256=C3WhAJfEoUasi2afHPuLpKiuRYixK-tc3j0-2Rw_g3E,5210
 firefighter/slack/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
@@ -482,6 +482,7 @@ firefighter_tests/test_incidents/test_models/test_incident_category.py,sha256=aR
 firefighter_tests/test_incidents/test_models/test_incident_model.py,sha256=AWyWfQYcHNP9GPizIo0wRxNGTJTEJnAwNSd4UmRq-dk,8626
 firefighter_tests/test_incidents/test_models/test_migrations/test_incident_migrations.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
 firefighter_tests/test_incidents/test_utils/test_date_utils.py,sha256=ogP7qOEwItL4YGI5gbQPVssOS9ilwiuZC8OrT2qngBY,6568
+firefighter_tests/test_incidents/test_views/test_incident_detail_jira_postmortem.py,sha256=WM1nr3XLGMqOEyr4e6XsWcU01aH4spiGnXd9YsVoVZY,1925
 firefighter_tests/test_incidents/test_views/test_incident_detail_view.py,sha256=lkCIRfz99Ea0o0Id08LFWrjXLDmHv6XvezaSsjg-eYQ,871
 firefighter_tests/test_incidents/test_views/test_index_view.py,sha256=InpxbaWOFwRn4YWeIKZhj17vMymrQQf2p2LFhe2Bcdw,816
 firefighter_tests/test_jira_app/__init__.py,sha256=JxZ3v-0kiHOoO-N3kR8NHTmD8tEvuEYKW1GX_S1ZLMY,33
@@ -493,14 +494,15 @@ firefighter_tests/test_jira_app/test_postmortem_issue_link.py,sha256=CF0UDmHOY08
 firefighter_tests/test_jira_app/test_postmortem_service.py,sha256=gUimkgf3NNIxHFDeZ5GNNzLHdaPKQCZDrTPdHmbyqDc,15120
 firefighter_tests/test_jira_app/test_timeline_template.py,sha256=_PtFnIib2HfjyylNRQXcjvdhrsoAJICOKauIDsYFQRk,4902
 firefighter_tests/test_raid/conftest.py,sha256=i_TOquYIMLDyVQ97uqxTqPJszVz4qq7L_Q7YJxTuS1o,4090
+firefighter_tests/test_raid/test_jira_status_sync.py,sha256=h0RTMAxL9paWXgF5zzlglqUNLfblmi-9dhrTZdh1ayk,7482
 firefighter_tests/test_raid/test_raid_alert_p4_p5.py,sha256=rz9orbt1E1vJ5POQyVZ6-SEPvqB55-xhwIWHicdfgDg,9356
 firefighter_tests/test_raid/test_raid_client.py,sha256=KTqELERpWno7XhF9LpabpxkHoJiWWrryUg5LHi5Yfjo,22456
 firefighter_tests/test_raid/test_raid_client_users.py,sha256=9uma1wBhaiCoG75XAZHqpT8oGTnqFJRMCi7a3XctNtM,3631
 firefighter_tests/test_raid/test_raid_forms.py,sha256=8hiXftYPO_lY0heKHqoreUW2s8AcedUme48wTq4hwNE,21931
 firefighter_tests/test_raid/test_raid_models.py,sha256=nq-fVClB_P24W8WrZruOPt8wlHUVGYI7wxJR7tH6AnM,5042
-firefighter_tests/test_raid/test_raid_serializers.py,sha256=xkMK9npvDMAr2pUpP4mFbLszrR6qyTzsJRd-uNOzEhM,22560
+firefighter_tests/test_raid/test_raid_serializers.py,sha256=6gjt8n8BYZwg6bXyuIieBQ9L13N4T_WFi3A0MES-Rpk,23186
 firefighter_tests/test_raid/test_raid_service.py,sha256=AqVyrRjW2tr0sfbXS4lGlJ7mcxB2ACEXAR8Bv0pXnj0,16755
-firefighter_tests/test_raid/test_raid_signals.py,sha256=QkGd3alp5O6t5kz66ukpNY-uZdIKJsqxvEPc8exORP4,8974
+firefighter_tests/test_raid/test_raid_signals.py,sha256=VrCBF3e-mEMkTVbqjNb7QJaQ5eQVVERJXBlCzFQTvFM,9210
 firefighter_tests/test_raid/test_raid_transitions.py,sha256=mtmMKwukxmZSM-R619BQ3Z_2AB-qY6imvDgUF0A3_tw,4784
 firefighter_tests/test_raid/test_raid_utils.py,sha256=i6JBwim1G-qynwxprNZekxl9K7Vis4FFvNkw3wT2jTM,1016
 firefighter_tests/test_raid/test_raid_views.py,sha256=paAhh4k2EDlmG1ehwNhMuYIhr1okqrvM7xlkaTAo2V0,6825
@@ -526,10 +528,10 @@ firefighter_tests/test_slack/views/modals/test_opening_unified.py,sha256=OejtLyc
 firefighter_tests/test_slack/views/modals/test_postmortem_modal.py,sha256=zNN40sIRSM5w_kyOcQ-AODkH5WpVxkSGVXkh9rMgmQ0,2378
 firefighter_tests/test_slack/views/modals/test_send_sos.py,sha256=_rE6jD-gOzcGyhlY0R9GzlGtPx65oOOguJYdENgxtLc,1289
 firefighter_tests/test_slack/views/modals/test_status.py,sha256=oQzPfwdg2tkbo9nfkO1GfS3WydxqSC6vy1AZjZDKT30,2226
-firefighter_tests/test_slack/views/modals/test_update_status.py,sha256=vbHGx6dkM_0swE1vJ0HrkhI1oJzD_WHZuIQ-_arAxXo,55686
+firefighter_tests/test_slack/views/modals/test_update_status.py,sha256=y33RKdjiPsN2hcKQB6tMbNwSMy8KMiZKjf4VdFb5_vc,56581
 firefighter_tests/test_slack/views/modals/test_utils.py,sha256=DJd2n9q6fFu8UuCRdiq9U_Cn19MdnC5c-ydLLrk6rkc,5218
-firefighter_incident-0.0.35.dist-info/METADATA,sha256=1BxWPGFrlMdWMWqRPIiN_4_ldSwsVU_kE-83ODpUkxQ,5570
-firefighter_incident-0.0.35.dist-info/WHEEL,sha256=WLgqFyCfm_KASv4WHyYy0P3pM_m7J5L9k2skdKLirC8,87
-firefighter_incident-0.0.35.dist-info/entry_points.txt,sha256=c13meJbv7YNmYz7MipMOQwzQ5IeFOPXUBYAJ44XMQsM,61
-firefighter_incident-0.0.35.dist-info/licenses/LICENSE,sha256=krRiGp-a9-1nH1bWpBEdxyTKLhjLmn6DMVVoIb0zF90,1087
-firefighter_incident-0.0.35.dist-info/RECORD,,
+firefighter_incident-0.0.37.dist-info/METADATA,sha256=JGhAeFgwEjVwqN25L93YHvwHWg62s-FZbo8haHkuhh4,5570
+firefighter_incident-0.0.37.dist-info/WHEEL,sha256=WLgqFyCfm_KASv4WHyYy0P3pM_m7J5L9k2skdKLirC8,87
+firefighter_incident-0.0.37.dist-info/entry_points.txt,sha256=c13meJbv7YNmYz7MipMOQwzQ5IeFOPXUBYAJ44XMQsM,61
+firefighter_incident-0.0.37.dist-info/licenses/LICENSE,sha256=krRiGp-a9-1nH1bWpBEdxyTKLhjLmn6DMVVoIb0zF90,1087
+firefighter_incident-0.0.37.dist-info/RECORD,,

firefighter_tests/test_incidents/test_views/test_incident_detail_jira_postmortem.py

@@ -0,0 +1,63 @@
+"""Test incident detail view displays Jira post-mortem links."""
+
+from __future__ import annotations
+
+import pytest
+from django.apps import apps
+from django.test import Client
+from django.urls import reverse
+
+from firefighter.incidents.models import Incident
+from firefighter.incidents.models.user import User
+
+pytestmark = pytest.mark.skipif(
+    not apps.is_installed("firefighter.jira_app"),
+    reason="Jira app not installed",
+)
+
+
+@pytest.mark.django_db
+def test_incident_detail_view_with_jira_postmortem(
+    client: Client, incident_saved: Incident, admin_user: User
+) -> None:
+    """Test that Jira post-mortem link is displayed when it exists."""
+    from firefighter.jira_app.models import JiraPostMortem
+
+    # Create a Jira post-mortem for the incident
+    JiraPostMortem.objects.create(
+        incident=incident_saved,
+        jira_issue_key="INCIDENT-123",
+        jira_issue_id="10001",
+        created_by=admin_user,
+    )
+
+    client.force_login(admin_user)
+    response = client.get(
+        reverse("incidents:incident-detail", args=[incident_saved.id])
+    )
+
+    assert response.status_code == 200
+    response_content = str(response.content)
+
+    # Check that the post-mortem section is present
+    assert "Post-mortem" in response_content
+
+    # Check that the Jira issue key is displayed
+    assert "INCIDENT-123" in response_content
+
+    # Check that the URL is correctly formed
+    assert "browse/INCIDENT-123" in response_content
+
+
+@pytest.mark.django_db
+def test_incident_detail_view_without_jira_postmortem(
+    client: Client, incident_saved: Incident, admin_user: User
+) -> None:
+    """Test that incident detail view works without Jira post-mortem."""
+    client.force_login(admin_user)
+    response = client.get(
+        reverse("incidents:incident-detail", args=[incident_saved.id])
+    )
+
+    assert response.status_code == 200
+    # Page should load successfully even without post-mortem