firefighter-incident 0.0.26__py3-none-any.whl → 0.0.28__py3-none-any.whl

firefighter/slack/views/modals/closure_reason.py

@@ -115,6 +115,19 @@ class ClosureReasonModal(IncidentSelectableModalMixin, SlackModal):
         self, ack: Ack, body: dict[str, Any], incident: Incident, user: User
     ) -> bool | None:
         """Handle the closure reason modal submission."""
+        # Extract form values up-front so validation can account for the submitted reason
+        state_values = body["view"]["state"]["values"]
+        closure_reason = state_values["closure_reason"]["select_closure_reason"][
+            "selected_option"
+        ]["value"]
+        closure_reference = (
+            state_values["closure_reference"]["input_closure_reference"].get(
+                "value", ""
+            )
+            or ""
+        )
+        message = state_values["closure_message"]["input_closure_message"]["value"]
+
         # For early closure (OPEN/INVESTIGATING), we bypass normal workflow checks
         # For normal closure (MITIGATED/POST_MORTEM), we must validate key events
         current_status = incident.status
@@ -134,22 +147,31 @@ class ClosureReasonModal(IncidentSelectableModalMixin, SlackModal):
                     }
                 )
                 return False
+        else:
+            # Early closure path - validate with the submitted closure reason
+            # Temporarily inject the submitted closure_reason so early-closure bypass applies
+            original_closure_reason = incident.closure_reason
+            incident.closure_reason = closure_reason
+            try:
+                can_close, reasons = incident.can_be_closed
+            finally:
+                incident.closure_reason = original_closure_reason
+            if not can_close:
+                # Build error message from reasons
+                error_messages = [reason[1] for reason in reasons]
+                error_text = "\n".join([f"• {msg}" for msg in error_messages])
+                ack(
+                    response_action="errors",
+                    errors={
+                        "closure_message": f"Cannot close this incident:\n{error_text}"
+                    },
+                )
+                return False
 
         # Clear ALL modals in the stack (not just this one)
         # This ensures the underlying "Update Status" modal is also closed
         ack(response_action="clear")
 
-        # Extract form values
-        state_values = body["view"]["state"]["values"]
-        closure_reason = state_values["closure_reason"]["select_closure_reason"][
-            "selected_option"
-        ]["value"]
-        closure_reference = (
-            state_values["closure_reference"]["input_closure_reference"].get("value", "")
-            or ""
-        )
-        message = state_values["closure_message"]["input_closure_message"]["value"]
-
         try:
             # Update incident with closure fields
             incident.closure_reason = closure_reason
@@ -165,9 +187,7 @@ class ClosureReasonModal(IncidentSelectableModalMixin, SlackModal):
             )
 
         except Exception:
-            logger.exception(
-                "Error closing incident #%s with reason", incident.id
-            )
+            logger.exception("Error closing incident #%s with reason", incident.id)
             respond(
                 body=body,
                 text=f"❌ Failed to close incident #{incident.id}",
@@ -182,7 +202,11 @@ class ClosureReasonModal(IncidentSelectableModalMixin, SlackModal):
                         f"✅ Incident #{incident.id} has been closed.\n"
                         f"*Reason:* {ClosureReason(closure_reason).label}\n"
                         f"*Message:* {message}"
-                        + (f"\n*Reference:* {closure_reference}" if closure_reference else "")
+                        + (
+                            f"\n*Reference:* {closure_reference}"
+                            if closure_reference
+                            else ""
+                        )
                     ),
                 )
             except SlackApiError as e:

firefighter/slack/views/modals/postmortem.py

@@ -3,10 +3,15 @@ from __future__ import annotations
 import logging
 from typing import TYPE_CHECKING, Any
 
-from slack_sdk.models.blocks.blocks import Block, SectionBlock
+from django.core.exceptions import ObjectDoesNotExist
+from slack_sdk.models.blocks.block_elements import ButtonElement
+from slack_sdk.models.blocks.blocks import ActionsBlock, Block, SectionBlock
 from slack_sdk.models.views import View
 
-from firefighter.slack.views.modals.base_modal.base import SlackModal
+from firefighter.confluence.models import PostMortemManager
+from firefighter.incidents.models.incident import Incident
+from firefighter.slack.utils import respond
+from firefighter.slack.views.modals.base_modal.base import SlackModal, app
 from firefighter.slack.views.modals.base_modal.mixins import (
     IncidentSelectableModalMixin,
 )
@@ -14,8 +19,6 @@ from firefighter.slack.views.modals.base_modal.mixins import (
 if TYPE_CHECKING:
     from slack_bolt.context.ack.ack import Ack
 
-    from firefighter.incidents.models.incident import Incident
-
 
 logger = logging.getLogger(__name__)
 
@@ -32,8 +35,8 @@ class PostMortemModal(
         blocks: list[Block] = []
 
         # Check existing post-mortems
-        has_confluence = hasattr(incident, "postmortem_for")
-        has_jira = hasattr(incident, "jira_postmortem_for")
+        has_confluence = _safe_has_relation(incident, "postmortem_for")
+        has_jira = _safe_has_relation(incident, "jira_postmortem_for")
 
         if has_confluence or has_jira:
             blocks.append(
@@ -53,12 +56,29 @@ class PostMortemModal(
                         text=f"• Jira: <{incident.jira_postmortem_for.issue_url}|{incident.jira_postmortem_for.jira_issue_key}>"
                     )
                 )
-        else:
+        elif incident.needs_postmortem:
             blocks.append(
                 SectionBlock(
                     text=f"Post-mortem for incident #{incident.id} will be automatically created when the incident reaches MITIGATED status."
                 )
             )
+        else:
+            blocks.extend(
+                [
+                    SectionBlock(
+                        text="P3 incident post-mortem is not mandatory. You can still have one if you think is necessary by clicking on the button below."
+                    ),
+                    ActionsBlock(
+                        elements=[
+                            ButtonElement(
+                                text="Create post-mortem now",
+                                action_id="incident_create_postmortem_now",
+                                value=str(incident.id),
+                            )
+                        ]
+                    ),
+                ]
+            )
 
         return View(
             type="modal",
@@ -76,4 +96,52 @@ class PostMortemModal(
         ack()
 
 
+@app.action("incident_create_postmortem_now")
+def handle_create_postmortem_action(ack: Ack, body: dict[str, Any]) -> None:
+    """Create post-mortem(s) on demand from the modal (e.g. P3+ incidents)."""
+    ack()
+
+    incident_id = str(body.get("actions", [{}])[0].get("value", "")).strip()
+    try:
+        incident = Incident.objects.get(pk=incident_id)
+    except Incident.DoesNotExist:
+        respond(body, text=":x: Incident not found.")
+        return
+
+    try:
+        confluence_pm, jira_pm = PostMortemManager.create_postmortem_for_incident(
+            incident
+        )
+    except Exception:
+        logger.exception("Failed to create post-mortem for incident #%s", incident_id)
+        respond(body, text=":x: Failed to create post-mortem. Please try again.")
+        return
+
+    created_targets: list[str] = []
+    if confluence_pm:
+        created_targets.append("Confluence")
+    if jira_pm:
+        created_targets.append("Jira")
+
+    if created_targets:
+        targets = " and ".join(created_targets)
+        respond(body, text=f":white_check_mark: {targets} post-mortem created.")
+    else:
+        respond(body, text=":warning: No post-mortem was created.")
+
+
 modal_postmortem = PostMortemModal()
+
+
+def _safe_has_relation(instance: Incident, attr: str) -> bool:
+    """Safely check if a reverse relation exists without triggering KeyError in cache.
+
+    Django's reverse OneToOne descriptor can raise KeyError when using hasattr
+    on unsaved or freshly created instances. We guard against that here.
+    """
+    try:
+        getattr(instance, attr)
+    except (AttributeError, ObjectDoesNotExist, KeyError):
+        return False
+    else:
+        return True

firefighter/slack/views/modals/update_status.py

@@ -127,12 +127,12 @@ class UpdateStatusModal(ModalForm[UpdateStatusFormSlack]):
                     # Build error message from reasons
                     error_messages = [reason[1] for reason in reasons]
                     error_text = "\n".join([f"• {msg}" for msg in error_messages])
-                    logger.warning(f"Cannot close incident #{incident.id} via Update Status: {error_text}")
+                    logger.warning(
+                        f"Cannot close incident #{incident.id} via Update Status: {error_text}"
+                    )
                     ack(
                         response_action="errors",
-                        errors={
-                            "status": f"Cannot close this incident:\n{error_text}"
-                        }
+                        errors={"status": f"{error_text}"},
                     )
                     return
 

{firefighter_incident-0.0.26.dist-info → firefighter_incident-0.0.28.dist-info}/METADATA

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: firefighter-incident
-Version: 0.0.26
+Version: 0.0.28
 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.26.dist-info → firefighter_incident-0.0.28.dist-info}/RECORD

@@ -6,7 +6,7 @@ gunicorn.conf.py,sha256=vHsTGjaKOr8FDMp6fTKYTX4AtokmPgYvvt5Mr0Q6APc,273
 main.py,sha256=CsbprHoOYhjCLpTJmq9Z_aRYFoFgWxoz2pDLuwm8Eqg,1558
 manage.py,sha256=5ivHGD13C6nJ8QvltKsJ9T9akA5he8da70HLWaEP3k8,689
 firefighter/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
-firefighter/_version.py,sha256=Fnf259POysYSI-aH6DhnxuHUxCVOFWsQLs4I5Ur_KMA,706
+firefighter/_version.py,sha256=rsZxmANBVfE-nN63y-ZZ_71Lkm6YP4u4TgVEBJV3mNM,706
 firefighter/api/__init__.py,sha256=JQW0Bv6xwGqy7ioxx3h6UGMzkkJ4DntDpbvV1Ncgi8k,136
 firefighter/api/admin.py,sha256=x9Ysy-GiYjb0rynmFdS9g56e6n24fkN0ouGy5QD9Yrc,4629
 firefighter/api/apps.py,sha256=P5uU1_gMrDfzurdMbfqw1Bnb2uNKKcMq17WBPg2sLhc,204
@@ -55,7 +55,7 @@ firefighter/confluence/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3
 firefighter/confluence/admin.py,sha256=aDXghuuLc7G_TLt-655M31smx-H6vkIgLtEmmNCA3lg,1490
 firefighter/confluence/apps.py,sha256=vKswBwQL7L9e2JQwvRb7xy3myyE_GRldYX78jSY3XCM,406
 firefighter/confluence/client.py,sha256=xjSsrsGPF75JANNvam2YgiUkztuXhOcs9pMmPbb7ymk,6361
-firefighter/confluence/models.py,sha256=ExFip6Tw199jI5LVLqC8t4pOPsvQxfEhtQeWCqTijFw,8769
+firefighter/confluence/models.py,sha256=66RNfD-lRdixZtOo5pNW2e-LZboTRjdoH2h-R7ne-q0,9154
 firefighter/confluence/serializers.py,sha256=CzuHVXIJNS47NCAJLXSTDOevtg5sf309XXEcWKQ1sAQ,258
 firefighter/confluence/service.py,sha256=dOQXj0uDInEm25nvL6lXiSH4hQ5oC2VDyBd1zbEcZ5U,12296
 firefighter/confluence/tables.py,sha256=ANEtFXzXyPK6E5FIrBC5XoQt5R3ZUY1DME_RbD1h_NE,732
@@ -147,6 +147,10 @@ firefighter/incidents/forms/update_key_events.py,sha256=1Xmnxe5OgZqLFS2HmMzQm3VG
 firefighter/incidents/forms/update_roles.py,sha256=Q26UPfwAj-8N23RNZLQkvmHGnS1_j_X5KQWjJmPjMKY,3635
 firefighter/incidents/forms/update_status.py,sha256=7GSno_EqD2Brd6wWcSb3zsP6nz8_mUTXXnl0QCRhv48,6682
 firefighter/incidents/forms/utils.py,sha256=15e_dBebVd9SvX03DYd0FyZ8s0YpxyBlZfIzEZattwg,4267
+firefighter/incidents/management/__init__.py,sha256=A2LtnedT5NvTcNAN5nXMkPwK56JBNLuptcyObvq7zcc,40
+firefighter/incidents/management/commands/__init__.py,sha256=wc5DFEklUo-wB-6VAAmsV5UTbo5s3t936Lu61z4lojs,29
+firefighter/incidents/management/commands/backdate_incident_mitigated.py,sha256=phAXH18TNvzA03o1XtJfRVeOPbrKp8wdsBMx-QGAIeo,3410
+firefighter/incidents/management/commands/test_postmortem_reminders.py,sha256=Bx-AVhkSjkL6c2_Eh7mRHa7qOEtytDl9T2hJYxcBC-4,4233
 firefighter/incidents/migrations/0001_initial_oss.py,sha256=OCrPbxf90h3NW9xolGGcsAryHKptD1TtKj5FucjBjg8,60311
 firefighter/incidents/migrations/0002_alter_severity_name_alter_user_password_featureteam.py,sha256=YfIJhw_-Yqm8qrkbp01461bkcUr7v5Zy90oHjkY3bSA,1113
 firefighter/incidents/migrations/0003_delete_featureteam.py,sha256=kH5UUSx3k5DtjR_goDxROdV0htCC2JZfBGwJpn-dEQs,336
@@ -176,12 +180,13 @@ firefighter/incidents/migrations/0026_alter_incidentcategory_options_and_more.py
 firefighter/incidents/migrations/0027_add_closure_fields.py,sha256=MDWckXmjJNC2iVoFJD6IIwDmmqyeL1VG_pHR568JAtk,1344
 firefighter/incidents/migrations/0028_add_closure_reason_constraint.py,sha256=z6FjCURDt9c-hyBeCvCKsbZOiuReYtbjtguIh3T6dnk,920
 firefighter/incidents/migrations/0029_add_custom_fields_to_incident.py,sha256=G6DsnP5bM4Hy0s8IqXhLYzFKt3eumEsCnJfPIw5tcX4,567
+firefighter/incidents/migrations/0030_add_mitigated_at_field.py,sha256=pELNJWbAuctv_dA-pdD3fsqg4qMm5f6upW9hFrjNLDI,546
 firefighter/incidents/migrations/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
 firefighter/incidents/models/__init__.py,sha256=FLVyBwIdyxLdgSvXRAKC3fry9YwwqlqhitTIuG0vWrk,877
 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=UdL9bc-3Ou-OS8P-nHO4YF6-Hk0mDkGNq1zjOq-TNFk,28022
+firefighter/incidents/models/incident.py,sha256=4G4vLurh2bgMRo5eOuYgKWNRC8Xf_07cFf_UdpLPOOg,29638
 firefighter/incidents/models/incident_category.py,sha256=g4OHv_XQhWcH6dvkqkyCgjlruo_1eih_CdtAPgPhaW4,7744
 firefighter/incidents/models/incident_cost.py,sha256=juwOfJKRaNQpOHkRUCHShDDba0FU98YjRPkU4I0ofAU,1346
 firefighter/incidents/models/incident_cost_type.py,sha256=wm8diry_VySJzIjC9M3Yavv2tYbvJgpN9UDb2gFRuH4,845
@@ -273,7 +278,7 @@ firefighter/jira_app/admin.py,sha256=ZHAAbhy0hm_DcklK59KMmid_ZiPn8n5V6g7cZCSNrpc
 firefighter/jira_app/apps.py,sha256=T6vHrQuMZHJoTth-xjy3CbNfPv6DyXgcR3PSMju2JS4,504
 firefighter/jira_app/client.py,sha256=qpMqNTjJUq5OqAxmwvVOE20uJe7kp737HSdsiqUu1G4,21982
 firefighter/jira_app/models.py,sha256=2zKy5VaKkhiHYA8Dukz8g0NTG82Qy5UHAHY9eMv67NE,3097
-firefighter/jira_app/service_postmortem.py,sha256=7VQwtVGCVZaVGfRxQXL19IM4hHrgHmItUbrsyy2gZlM,10764
+firefighter/jira_app/service_postmortem.py,sha256=tqJN91vZPX5ISd_PAGWDebHK225eZvDvwhi7ONy_D-A,11325
 firefighter/jira_app/types.py,sha256=Ukak1U1EhcH2jQPN-UoEL6AMZ-kzPsQ8c7FUr7GmahE,956
 firefighter/jira_app/utils.py,sha256=3xuzr8viZCBm6j2J9oFzA4bUvVW8TN1DOdlpbruJ_TE,3443
 firefighter/jira_app/management/__init__.py,sha256=wy4qMZb7_K-INwwGGEhMtEeI0XTLqgUw4P8_-VEnrEw,40
@@ -283,10 +288,10 @@ firefighter/jira_app/migrations/0002_add_jira_postmortem_model.py,sha256=oFSbYNc
 firefighter/jira_app/migrations/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
 firefighter/jira_app/signals/__init__.py,sha256=OpXFlbRgIrh73DGlUzQ6WUTedKsD5wYW9bxGMq_DnIs,325
 firefighter/jira_app/signals/incident_key_events_updated.py,sha256=uaV3MON1QzeOZizzAwSdyktBwe2mWxHJeNSvy9MYc3k,3204
-firefighter/jira_app/signals/postmortem_created.py,sha256=eOTQKcjspL-JHvXiCu-U1FzrqIr6OJ20GOK6gJ4dojw,5904
+firefighter/jira_app/signals/postmortem_created.py,sha256=S7sKbEgo5RroC5ji1OepAT3HMckwS120dpKeUteNaXA,8300
 firefighter/jira_app/tasks/__init__.py,sha256=XLCPkolM6LwIUGv0MNbk_0lCuBHyzgRFHsE3vTRD5ds,86
 firefighter/jira_app/tasks/sync_users_jira.py,sha256=sSSLsVCdzkPNRS6Gt8j0YwCTuoRqkJAJLxDBu7IElmM,1437
-firefighter/jira_app/templates/jira/postmortem/impact.txt,sha256=eYlX0rytUiKZZIrWxMN23QXkLZ8JwlwPe5S2oiqZyFA,259
+firefighter/jira_app/templates/jira/postmortem/impact.txt,sha256=WeF6uuwKjSER-CYqpLgH6p-wIkWgtVaeGKp8TsWSekQ,515
 firefighter/jira_app/templates/jira/postmortem/incident_summary.txt,sha256=Bnias41O8TR2v0CAWpOoyRVVBAyx6vk2iICeQsLOuCs,392
 firefighter/jira_app/templates/jira/postmortem/mitigation_actions.txt,sha256=7DWOcMhU0NiJugiUmEvSg1Z3ajW7IDZejprxt8urue0,242
 firefighter/jira_app/templates/jira/postmortem/root_causes.txt,sha256=27DgQdrtHvHcju1llyYqU1jufegeHjJN_qvVkyECINM,291
@@ -345,7 +350,7 @@ firefighter/slack/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU
 firefighter/slack/admin.py,sha256=pNJbA-szxUUrghxv_Z0BNezu6lULDzFcOu_K5i4m7Cs,13963
 firefighter/slack/apps.py,sha256=gR0zWTtqT58tjPayBX22ZSzMkLiNpmoOvLShNvhJA6Q,664
 firefighter/slack/factories.py,sha256=tnrUTbtgehCuBr24MtTyJ3uezKC6gJbOdHuYZ5JBoyU,3886
-firefighter/slack/rules.py,sha256=PHbqi8sSTZi60TaahLfwdVr1B3GG_Rz1ldAT5q3XvG4,1651
+firefighter/slack/rules.py,sha256=Y-DYJ_1D13a4nQNESCbhalNe2nC9xVNpYtSeWTgDcYc,2374
 firefighter/slack/slack_app.py,sha256=mvaH0hPFaNIUxEB7J0fy6y-PNPGsdPqjXFVmpTQ_hCo,4201
 firefighter/slack/slack_incident_context.py,sha256=PjE7-w-pGFyV4faw8EMsEFp4RG_T251RhofmqrsDG7Q,7277
 firefighter/slack/slack_templating.py,sha256=rWe8m1n648wizw08U_vLz8daRnp4zmkcWRqocIBpQj4,3841
@@ -359,7 +364,7 @@ firefighter/slack/management/commands/generate_manifest.py,sha256=zFWHAC7ioozcDd
 firefighter/slack/management/commands/switch_test_users.py,sha256=2KTSvCBxsEvZa61J8p0r3huPNhwuytcj2J7IawwZWpQ,11064
 firefighter/slack/messages/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
 firefighter/slack/messages/base.py,sha256=biH-YEAaldJ-OLHEs5ZjW-gtUYUbjOqxrAEflqV2XS0,4593
-firefighter/slack/messages/slack_messages.py,sha256=xvkV_sEbBRkyqGigR_oQeC6Cr9KLWqqwxDXmPywyEok,35553
+firefighter/slack/messages/slack_messages.py,sha256=9-zbxBuzDweKQZiwc2DMdiQcsnd23CGCi4YWhGgJx60,43713
 firefighter/slack/migrations/0001_initial_oss.py,sha256=XmTPgq7zCME2xDwzRFoVi4OegSIG9eSKoyTNoW05Qtg,12933
 firefighter/slack/migrations/0002_usergroup_tag.py,sha256=098tmGA81mT-R2uhb6uQfZ7gKiRG9bFhEwQ8rrp4SKM,583
 firefighter/slack/migrations/0003_alter_usergroup_tag.py,sha256=ncH3KUWEPZHlbdcAtOJ0KGt5H6EX-cKspTGU3osrAhE,591
@@ -368,6 +373,7 @@ firefighter/slack/migrations/0005_add_incident_categories_fields.py,sha256=KMdKf
 firefighter/slack/migrations/0006_copy_components_to_incident_categories.py,sha256=xUF7lLyWERux6SyIYHK2Uk1Yb4QLCGTaHW_KXVqX8n4,2478
 firefighter/slack/migrations/0007_remove_components_fields.py,sha256=_GXmcpB3enpVBT1NZ-tGDlh16r_cM-JkH2gebrmwIOs,563
 firefighter/slack/migrations/0008_alter_conversation_incident_categories_and_more.py,sha256=yzuAnunYvlF-wcYd7oe5h-kL5aOoawSXv_QGfmTFoBo,1034
+firefighter/slack/migrations/0009_add_postmortem_reminder_periodic_task.py,sha256=Vze5TvhQExaF7-KymByzMZZE1wa0GHlWZXP1v1hub5w,2035
 firefighter/slack/migrations/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
 firefighter/slack/models/__init__.py,sha256=MGc4yuDnVhmAiHy1-5rjaLIfVv9JOup5arRutcUs8Ak,332
 firefighter/slack/models/conversation.py,sha256=f7a0muD0lrpf8mIhF6E2gEhNsgwZFw9jlKIQppZhNL0,16227
@@ -388,6 +394,7 @@ firefighter/slack/tasks/__init__.py,sha256=28QxZkakyi9l7Ae83fQuzOS-9EaBiwuh_peUZ
 firefighter/slack/tasks/fetch_conversations_members.py,sha256=lLQ491_l8HEJrjoDpD0AETqoFUFogkyMJ002hguA-Dg,5381
 firefighter/slack/tasks/reminder_postmortem.py,sha256=mZvT4cpzmMhC6JrWhZb1uFvTJJrbkEKgcCy6liKHrKM,2322
 firefighter/slack/tasks/send_message.py,sha256=N0FIE93bUnzbHdWkSWC-4-eLn737u6hHq8F8erMu8kI,810
+firefighter/slack/tasks/send_postmortem_reminders.py,sha256=uyF9v8um2uxuYFyDSe2GhmG3iX0iAJrG1yE3LS0_NSo,4741
 firefighter/slack/tasks/send_reminders.py,sha256=hy1Q_rG2RUQdXNYEYiLyLnT7rkG8PFOmxur62YCCDrk,4370
 firefighter/slack/tasks/sync_users.py,sha256=T5ytYnZpcUqrh4sOklxWttsUk82C_2bwayg_fdcdg1g,2391
 firefighter/slack/tasks/update_usergroups_members.py,sha256=W-rPt3r2c9UboVMNiyQFvi_W7XWrR8ireVBWcCJMj5A,4642
@@ -410,20 +417,20 @@ firefighter/slack/views/events/message.py,sha256=c8tvo0btOUu_5Bc83oiO3IQbaEyoRiU
 firefighter/slack/views/events/message_deleted.py,sha256=tyA1-sAlG9ImcKIhqSn6EgujHmbvj4Uw2QzQ4JH4QwI,747
 firefighter/slack/views/events/reaction_added.py,sha256=AipwBnrU5B35D97YIZCXdSW8W7-9QTIIQqUcrLTLQ5c,4241
 firefighter/slack/views/modals/__init__.py,sha256=U9PapAIlpuYqBonOUmBGWT8_HjQa35ilMQJXGaFLgd0,1945
-firefighter/slack/views/modals/close.py,sha256=4j5iA-lmIFuCz7B9pgDmjxrqmfWFysqWEn1YIsE75zc,12161
-firefighter/slack/views/modals/closure_reason.py,sha256=N-gp0E6W8Z1d4aH1-8BxizNTlZ4syNIF4l5B9WZHFFA,7898
+firefighter/slack/views/modals/close.py,sha256=eTT1IVEMXUrdUXsu5VSmC6-cGUsOYUGG7CN-3yAnuDo,17482
+firefighter/slack/views/modals/closure_reason.py,sha256=qgjg6x5JBh5ggR6Z6aVhpnc3k8En38UxKtFb5RkTU4U,9016
 firefighter/slack/views/modals/downgrade_workflow.py,sha256=cRWsm3DmKRRI1-Jpjprb5xeY2U7HvRo6eZlUbGuzr1A,3192
 firefighter/slack/views/modals/edit.py,sha256=1N0OBSxsDuN6lJoH-djbEljy7f0LcDEpJF-U5YoEFXA,5895
 firefighter/slack/views/modals/key_event_message.py,sha256=C6yhQLQ6jBuhIr-YAoAyt-qZKu0V6nJMGZ_t3DLtUbo,5943
 firefighter/slack/views/modals/open.py,sha256=YIxpo8_C4cWCy_pQ3YRWl7NMyLmjqNjggTQINTBW6mo,29189
-firefighter/slack/views/modals/postmortem.py,sha256=7h_at4oMVFxN3IISbk1yzBDC46g4CFDsRnc873d8K-E,2451
+firefighter/slack/views/modals/postmortem.py,sha256=Re4F0ZQEEOfdXljVCkswU1ESZksmpAqRy3es3_Wmeiw,5070
 firefighter/slack/views/modals/select.py,sha256=Y-Ji_ALnzhYkXDBAyi497UL1Xn2vCGqXCtj8eog75Jk,3312
 firefighter/slack/views/modals/send_sos.py,sha256=bP6HgYyDwPrIcTq7n_sQz6UQsxhYbvBDS4HjM0uRccA,4838
 firefighter/slack/views/modals/status.py,sha256=C8-eJRtquSeaHe568SC7yCFef1k14m2_6lUqBezdSH8,3970
 firefighter/slack/views/modals/trigger_oncall.py,sha256=h_LAD5X5rjMFWiDYTEp5VB9OaF7sTvKZhNaW3KQkw5M,5065
 firefighter/slack/views/modals/update.py,sha256=OF9sf-Z6IiviNmjN28MQNYiUbJ5tha0MdHUQyPpVFiY,2150
 firefighter/slack/views/modals/update_roles.py,sha256=De3Gv67MZQHyNdonX3S99F5MtKF_Rj3y71gdWibxBaM,2419
-firefighter/slack/views/modals/update_status.py,sha256=kJYcyruEwo7hok-z4Ge8HPxLMc-XPm-OrB7X_faNd7U,6000
+firefighter/slack/views/modals/update_status.py,sha256=SYjQLXnWSZLk461b-L9WFRSxy7clIA4O0C4ZMrNWiuc,5964
 firefighter/slack/views/modals/utils.py,sha256=zKLJD2KhTGcX2d9WCYwshYRa6ok_9-ED1_pgOLp028s,2133
 firefighter/slack/views/modals/base_modal/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
 firefighter/slack/views/modals/base_modal/base.py,sha256=7mvOxZTtegSmitSMnDvu8BK0qLUXoudUsda6CaLjdkY,12479
@@ -459,7 +466,7 @@ firefighter_tests/test_firefighter/test_logging.py,sha256=4HUH73vLDwmOCpMiXwDasM
 firefighter_tests/test_firefighter/test_sso.py,sha256=uX2ry0REDgXzQc9Y1BmAgI0OgbmzWoOv9H_GDyOqQmQ,5205
 firefighter_tests/test_firefighter/test_urls.py,sha256=UMGx4oW98RoL0ceePkIIKEVjbHdFECvQuGNXYAJForQ,4839
 firefighter_tests/test_incidents/test_enums.py,sha256=wMxxL1uakrmzJIi-2xkAvG-Y3NDDmIt0PHyOAJBz0yQ,4341
-firefighter_tests/test_incidents/test_incident_urls.py,sha256=j663qeBkv31aMlA0sLfzBOmp0iEv9OB-xvIwzrjlSZk,3793
+firefighter_tests/test_incidents/test_incident_urls.py,sha256=VD9dj0IHHKXJHC5ApZg-L9CMtotaQb9uRAKcZIhOrDI,3978
 firefighter_tests/test_incidents/test_forms/conftest.py,sha256=YYF5Lm-Jmt-HM9zt_gjrNkiuqOaNMW8lLBr1crAP6J8,5423
 firefighter_tests/test_incidents/test_forms/test_closure_reason.py,sha256=H6RObqazFAit_pvo7N-lotiSsLOYMafZIk23A5Wiodg,3533
 firefighter_tests/test_incidents/test_forms/test_form_select_impact.py,sha256=DTaPGrJi8mXHfh7mhvDTKYVvDCxqarILauE59UDlwqo,3210
@@ -471,10 +478,10 @@ firefighter_tests/test_incidents/test_forms/test_update_key_events.py,sha256=rHR
 firefighter_tests/test_incidents/test_forms/test_update_status_workflow.py,sha256=q0xXU2BbBG8B0uvvyBWlo4HM8ckbcNAP05Fq8oJNtOw,16270
 firefighter_tests/test_incidents/test_forms/test_workflow_transitions.py,sha256=priKh7QYZxGDPu2SvPC8pGnqOsZWg5cLkyC40pDvLAU,7184
 firefighter_tests/test_incidents/test_models/test_incident_category.py,sha256=aRoBOhb8fNjLF9CMPZ1FXM8AT51Cd80XPsY2Y3wHY_M,5701
-firefighter_tests/test_incidents/test_models/test_incident_model.py,sha256=P5tmU5X4grt-yvoqjfuBkbd5tAWX3TptN-b_R3-1a9A,4719
+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_view.py,sha256=gKKFWIZVrD_P4p6DJjeHCW5uGXBUBVlCd95gJJYDpWQ,680
+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
 firefighter_tests/test_jira_app/conftest.py,sha256=HmZd7EBZgng-rb3kIaB14TPVMixMG4YEvnShVqgjodE,545
@@ -502,24 +509,25 @@ firefighter_tests/test_slack/test_conversation_tags.py,sha256=nNqTZRRBfF6Z4wpFSY
 firefighter_tests/test_slack/test_signals_downgrade.py,sha256=mgl4H5vwr2kImf6g4IZbhv7YEPmMzbYSaVr8E6taL88,5420
 firefighter_tests/test_slack/test_slack_utils.py,sha256=9PLobMNXh3xDyFuwzcQFpKJhe4j__sIgf_WRHIpANJw,3957
 firefighter_tests/test_slack/messages/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
-firefighter_tests/test_slack/messages/test_slack_messages.py,sha256=rs7zeBa_xsAyZIfXbOI1RA-zgViEPT5ZFNw6paBJvxc,17299
+firefighter_tests/test_slack/messages/test_slack_messages.py,sha256=uyxfeAy1BQxx1zcCzlSJWn5YF1EnH-5Kt2XoIn9dekM,17484
 firefighter_tests/test_slack/test_models/test_conversations.py,sha256=t3ttmgwiu7c-N55iU3XZPmrkEhvkTzJoXszJncy4Bts,793
 firefighter_tests/test_slack/test_models/test_incident_channel.py,sha256=qWoGe9iadmK6-R8usWvjH87AHRkvhG_dHQeC3kHeJrs,17487
 firefighter_tests/test_slack/test_models/test_slack_user.py,sha256=uzur-Rf03I5dpUTO4ZI6O1arBUrAorg1Zvgshf8M-J4,7000
 firefighter_tests/test_slack/views/modals/conftest.py,sha256=TKJVQgqWaFs3Gg1T526pti9XpZBtQs47WBH6L_qSDeo,4532
-firefighter_tests/test_slack/views/modals/test_close.py,sha256=wuqMkpyUWKvC_gaTnJvRmdm8-wsmbLpCU_ith4ihsLY,45447
-firefighter_tests/test_slack/views/modals/test_closure_reason_modal.py,sha256=hKfBv8fHfObmZc7OL6NUnxFueA1bowkNbu_-MF1anYI,5615
+firefighter_tests/test_slack/views/modals/test_close.py,sha256=FWNV7RIUpqp3tiz9IBbBxaZk1XQt2f7vWB5TzJKYK3o,45630
+firefighter_tests/test_slack/views/modals/test_closure_reason_modal.py,sha256=mvg5RiCXQEp1GhyOBCNW4idkNR1StgZjPvFrjzJ549Q,8333
 firefighter_tests/test_slack/views/modals/test_edit.py,sha256=ykirry-S3i6PtoSs3rff_k6jqmvv1oMWC_iR8e5Jsg0,12022
 firefighter_tests/test_slack/views/modals/test_form_utils_multiple_choice.py,sha256=Svab_ZyYTMf0T-uJEQcm7gS1WzxtC4gPh1W--Z2v_Y8,8415
 firefighter_tests/test_slack/views/modals/test_key_event_message.py,sha256=BCg-c27ZLJqNgFuG4JDgXrSTp8_sT4FeBtpASzSq8NI,1107
 firefighter_tests/test_slack/views/modals/test_open.py,sha256=IzgG9le5NN_CvltehAIqkj94ioTKCqdA6yoRp2NlNsE,10700
 firefighter_tests/test_slack/views/modals/test_opening_unified.py,sha256=OejtLyc_mehav2TDaLzUnhilMNvhCzc6T4FodCqfQPk,17406
+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=3ARHZPs22FTx7IjgOldzEpVxxWeHqEbe4kQphUuSp34,55928
+firefighter_tests/test_slack/views/modals/test_update_status.py,sha256=vbHGx6dkM_0swE1vJ0HrkhI1oJzD_WHZuIQ-_arAxXo,55686
 firefighter_tests/test_slack/views/modals/test_utils.py,sha256=DJd2n9q6fFu8UuCRdiq9U_Cn19MdnC5c-ydLLrk6rkc,5218
-firefighter_incident-0.0.26.dist-info/METADATA,sha256=6maVl16hOTcZXnEYTy1PhaP4ligQTcK5QVZesD6C5a0,5570
-firefighter_incident-0.0.26.dist-info/WHEEL,sha256=WLgqFyCfm_KASv4WHyYy0P3pM_m7J5L9k2skdKLirC8,87
-firefighter_incident-0.0.26.dist-info/entry_points.txt,sha256=c13meJbv7YNmYz7MipMOQwzQ5IeFOPXUBYAJ44XMQsM,61
-firefighter_incident-0.0.26.dist-info/licenses/LICENSE,sha256=krRiGp-a9-1nH1bWpBEdxyTKLhjLmn6DMVVoIb0zF90,1087
-firefighter_incident-0.0.26.dist-info/RECORD,,
+firefighter_incident-0.0.28.dist-info/METADATA,sha256=mM6LIiFAeLibO2jnQEf9KZiEOFlZJT09A3jrCExS5sM,5570
+firefighter_incident-0.0.28.dist-info/WHEEL,sha256=WLgqFyCfm_KASv4WHyYy0P3pM_m7J5L9k2skdKLirC8,87
+firefighter_incident-0.0.28.dist-info/entry_points.txt,sha256=c13meJbv7YNmYz7MipMOQwzQ5IeFOPXUBYAJ44XMQsM,61
+firefighter_incident-0.0.28.dist-info/licenses/LICENSE,sha256=krRiGp-a9-1nH1bWpBEdxyTKLhjLmn6DMVVoIb0zF90,1087
+firefighter_incident-0.0.28.dist-info/RECORD,,

firefighter_tests/test_incidents/test_incident_urls.py

@@ -4,6 +4,7 @@ import logging
 from typing import TYPE_CHECKING
 
 import pytest
+from django.apps import apps
 from django.urls import reverse
 
 from firefighter.incidents.factories import IncidentFactory, UserFactory
@@ -15,6 +16,9 @@ if TYPE_CHECKING:
 
 logger = logging.getLogger(__name__)
 
+if not apps.is_installed("firefighter.confluence"):
+    pytest.skip("Confluence app not installed; skipping incident URLs tests", allow_module_level=True)
+
 
 @pytest.mark.django_db
 def test_incidents_dashboard_unauthorized(client: Client) -> None:

firefighter_tests/test_incidents/test_models/test_incident_model.py

@@ -1,6 +1,7 @@
 from __future__ import annotations
 
 from typing import TYPE_CHECKING
+from unittest.mock import PropertyMock, patch
 
 import pytest
 from hypothesis import given
@@ -10,8 +11,11 @@ from hypothesis.strategies import builds
 from firefighter.incidents.enums import ClosureReason, IncidentStatus
 from firefighter.incidents.factories import IncidentFactory
 from firefighter.incidents.models import IncidentUpdate
+from firefighter.jira_app.models import JiraPostMortem
 
 if TYPE_CHECKING:
+    from pytest_mock import MockerFixture
+
     from firefighter.incidents.models import Incident
 
 
@@ -58,7 +62,9 @@ class TestIncidentCanBeClosed:
         can_close, reasons = incident.can_be_closed
 
         # Should be closable (assuming no missing milestones)
-        assert can_close is True or "STATUS_NOT_MITIGATED" not in [r[0] for r in reasons]
+        assert can_close is True or "STATUS_NOT_MITIGATED" not in [
+            r[0] for r in reasons
+        ]
 
     def test_can_close_incident_with_closure_reason(self) -> None:
         """Test that incidents with closure_reason can always be closed."""
@@ -73,6 +79,108 @@ class TestIncidentCanBeClosed:
         assert can_close is True
         assert reasons == []
 
+    def test_cannot_close_when_jira_postmortem_not_ready(self, settings: None) -> None:
+        """Block closure if Jira post-mortem exists but is not in Ready status."""
+        settings.ENABLE_JIRA_POSTMORTEM = True
+        incident = IncidentFactory.create(
+            _status=IncidentStatus.POST_MORTEM,
+            priority__value=1,
+            priority__needs_postmortem=True,
+            environment__value="PRD",
+        )
+        JiraPostMortem.objects.create(
+            incident=incident,
+            jira_issue_key="INC-999",
+            jira_issue_id="999",
+            created_by=incident.created_by,
+        )
+        incident.refresh_from_db()
+        assert hasattr(incident, "jira_postmortem_for")
+
+        with (
+            patch.object(
+                type(incident),
+                "needs_postmortem",
+                new_callable=PropertyMock,
+                return_value=True,
+            ),
+            patch.object(type(incident), "missing_milestones", return_value=[]),
+            patch(
+                "firefighter.jira_app.service_postmortem.jira_postmortem_service.is_postmortem_ready",
+                return_value=(False, "In Progress"),
+            ),
+        ):
+            can_close, reasons = incident.can_be_closed
+
+        assert can_close is False
+        assert any(r[0] == "POSTMORTEM_NOT_READY" for r in reasons)
+
+    def test_postmortem_ready_allows_closure(
+        self, mocker: MockerFixture, settings: None
+    ) -> None:
+        """When Jira PM is Ready, can_be_closed should allow closure for PM incidents."""
+        settings.ENABLE_JIRA_POSTMORTEM = True
+        incident = IncidentFactory.create(
+            _status=IncidentStatus.POST_MORTEM,
+            priority__value=1,
+            priority__needs_postmortem=True,
+            environment__value="PRD",
+        )
+        JiraPostMortem.objects.create(
+            incident=incident,
+            jira_issue_key="INC-READY",
+            jira_issue_id="123",
+            created_by=incident.created_by,
+        )
+
+        mocker.patch.object(type(incident), "missing_milestones", return_value=[])
+        mocker.patch(
+            "firefighter.jira_app.service_postmortem.jira_postmortem_service.is_postmortem_ready",
+            return_value=(True, "Ready"),
+        )
+
+        can_close, reasons = incident.can_be_closed
+
+        assert can_close is True
+        assert reasons == []
+
+    def test_postmortem_status_unknown_sets_reason(
+        self, mocker: MockerFixture, settings: None
+    ) -> None:
+        """Errors while checking Jira PM should return POSTMORTEM_STATUS_UNKNOWN."""
+        settings.ENABLE_JIRA_POSTMORTEM = True
+        incident = IncidentFactory.create(
+            _status=IncidentStatus.POST_MORTEM,
+            priority__value=1,
+            priority__needs_postmortem=True,
+            environment__value="PRD",
+        )
+        JiraPostMortem.objects.create(
+            incident=incident,
+            jira_issue_key="INC-ERR",
+            jira_issue_id="124",
+            created_by=incident.created_by,
+        )
+        incident.refresh_from_db()
+        assert hasattr(incident, "jira_postmortem_for")
+
+        mocker.patch.object(type(incident), "missing_milestones", return_value=[])
+        mocker.patch(
+            "firefighter.jira_app.service_postmortem.jira_postmortem_service.is_postmortem_ready",
+            side_effect=Exception("boom"),
+        )
+        mocker.patch.object(
+            type(incident),
+            "needs_postmortem",
+            new_callable=PropertyMock,
+            return_value=True,
+        )
+
+        can_close, reasons = incident.can_be_closed
+
+        assert can_close is False
+        assert any(r[0] == "POSTMORTEM_STATUS_UNKNOWN" for r in reasons)
+
 
 @pytest.mark.django_db
 class TestIncidentSetStatus:

firefighter_tests/test_incidents/test_views/test_incident_detail_view.py

@@ -1,12 +1,16 @@
 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
 
+if not apps.is_installed("firefighter.confluence"):
+    pytest.skip("Confluence app not installed; skipping incident detail view test", allow_module_level=True)
+
 
 @pytest.mark.django_db
 def test_incident_detail_view(

firefighter_tests/test_slack/messages/test_slack_messages.py

@@ -1,6 +1,7 @@
 from __future__ import annotations
 
 import pytest
+from django.apps import apps
 
 from firefighter.incidents.enums import IncidentStatus
 from firefighter.incidents.factories import IncidentFactory, UserFactory
@@ -18,6 +19,9 @@ try:
 except (ImportError, AttributeError):
     PostMortem = None
 
+if not apps.is_installed("firefighter.confluence"):
+    pytest.skip("Confluence app not installed; skipping slack message tests", allow_module_level=True)
+
 
 @pytest.mark.django_db
 class TestSlackMessageIncidentStatusUpdated:

firefighter_tests/test_slack/views/modals/test_close.py

@@ -5,6 +5,7 @@ from copy import deepcopy
 from unittest.mock import MagicMock, PropertyMock
 
 import pytest
+from django.apps import apps
 from pytest_mock import MockerFixture
 
 from firefighter.incidents.enums import IncidentStatus
@@ -14,6 +15,9 @@ from firefighter.slack.views import CloseModal
 
 logger = logging.getLogger(__name__)
 
+if not apps.is_installed("firefighter.confluence"):
+    pytest.skip("Confluence app not installed; skipping close modal tests", allow_module_level=True)
+
 
 @pytest.mark.django_db
 class TestCloseModal: