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

firefighter/_version.py

@@ -28,7 +28,7 @@ version_tuple: VERSION_TUPLE
 commit_id: COMMIT_ID
 __commit_id__: COMMIT_ID
 
-__version__ = version = '0.0.27'
-__version_tuple__ = version_tuple = (0, 0, 27)
+__version__ = version = '0.0.28'
+__version_tuple__ = version_tuple = (0, 0, 28)
 
 __commit_id__ = commit_id = None

firefighter/jira_app/templates/jira/postmortem/impact.txt

@@ -1,12 +1,17 @@
 h2. Impact
 
+h3. Business Impact
+
+*{color:red}_TODO: Describe the business impact and precise the BV daily loss._{color}*
+*{color:red}_For the BV daily loss, use [this documentation|https://manomano.atlassian.net/wiki/spaces/TC/pages/4089315451/How+to+calculate+BV+loss]._{color}*
+
+h3. User Impact
+
+*{color:red}_TODO: Describe the impact on users._{color}*
+
 h3. Affected Systems
 {% if components %}
 {% for component in components %}* {{ component.name }}
 {% endfor %}
 {% else %}_No components recorded._
 {% endif %}
-
-h3. User Impact
-
-*{color:red}_TODO: Describe the impact on users and business._{color}*

firefighter/slack/messages/slack_messages.py

@@ -109,15 +109,36 @@ class SlackMessageIncidentPostMortemReminder(SlackMessageSurface):
                     action_id=UpdateStatusModal.open_action,
                 ),
             ),
-            SectionBlock(
-                text="2. Edit your post-mortem on Confluence",
-                accessory=ButtonElement(
-                    text="Edit post-mortem",
-                    value=self.incident.postmortem_for.page_edit_url,
-                    url=self.incident.postmortem_for.page_edit_url,
-                    action_id="open_link",
-                ),
-            ),
+        ]
+
+        # Add post-mortem editing options based on available services
+        if hasattr(self.incident, "postmortem_for") and self.incident.postmortem_for:
+            blocks.append(
+                SectionBlock(
+                    text="2. Edit your post-mortem on Confluence",
+                    accessory=ButtonElement(
+                        text="Edit post-mortem",
+                        value=self.incident.postmortem_for.page_edit_url,
+                        url=self.incident.postmortem_for.page_edit_url,
+                        action_id="open_link",
+                    ),
+                )
+            )
+        elif hasattr(self.incident, "jira_postmortem_for") and self.incident.jira_postmortem_for:
+            jira_pm = self.incident.jira_postmortem_for
+            blocks.append(
+                SectionBlock(
+                    text="2. Edit your post-mortem on Jira",
+                    accessory=ButtonElement(
+                        text=f"Edit Jira post-mortem ({jira_pm.jira_issue_key})",
+                        url=jira_pm.issue_url,
+                        action_id="open_link",
+                    ),
+                )
+            )
+
+        # Continue with remaining steps
+        blocks.extend([
             SectionBlock(
                 text=f"3. Submit the key events to {APP_DISPLAY_NAME}",
                 **accessory_kwargs,
@@ -138,7 +159,8 @@ class SlackMessageIncidentPostMortemReminder(SlackMessageSurface):
                     )
                 ]
             ),
-        ]
+        ])
+
         if POSTMORTEM_HELP_URL:
             blocks.insert(
                 4,
@@ -184,14 +206,30 @@ class SlackMessageIncidentFixedNextActions(SlackMessageSurface):
                 ),
             ),
             DividerBlock(),
-            ContextBlock(
-                elements=[
-                    MarkdownTextObject(
-                        text="A post-mortem is *not* required for this incident.\nIf you want to create one, use `/incident postmortem` to create a new post-mortem page on Confluence."
-                    )
-                ]
-            ),
         ]
+
+        # Add post-mortem context message if any post-mortem service is enabled
+        enable_confluence = getattr(settings, "ENABLE_CONFLUENCE", False)
+        enable_jira_postmortem = getattr(settings, "ENABLE_JIRA_POSTMORTEM", False)
+
+        if enable_confluence or enable_jira_postmortem:
+            postmortem_text = "A post-mortem is *not* required for this incident.\nIf you want to create one, use `/incident postmortem` to create a new post-mortem page"
+
+            if enable_confluence and enable_jira_postmortem:
+                postmortem_text += " on Confluence or Jira."
+            elif enable_confluence:
+                postmortem_text += " on Confluence."
+            elif enable_jira_postmortem:
+                postmortem_text += " on Jira."
+
+            blocks.append(
+                ContextBlock(
+                    elements=[
+                        MarkdownTextObject(text=postmortem_text)
+                    ]
+                )
+            )
+
         return blocks
 
 
@@ -612,7 +650,23 @@ class SlackMessageIncidentPostMortemCreated(SlackMessageSurface):
         return "\n".join(parts)
 
     def get_blocks(self) -> list[Block]:
-        return [SectionBlock(text=self.get_text())]
+        blocks: list[Block] = [SectionBlock(text=self.get_text())]
+
+        # Add documentation link if Jira post-mortem exists
+        if hasattr(self.incident, "jira_postmortem_for") and self.incident.jira_postmortem_for:
+            blocks.append(
+                SectionBlock(
+                    text="Need guidance on how to fill Post-Mortems in Jira? See our documentation",
+                    accessory=ButtonElement(
+                        text="Open documentation",
+                        url="https://manomano.atlassian.net/wiki/spaces/TC/pages/5639635000/How+to+fill+Post-Mortems+in+Jira",
+                        value="jira_postmortem_documentation",
+                        action_id="open_link",
+                    ),
+                )
+            )
+
+        return blocks
 
 
 class SlackMessageIncidentPostMortemCreatedAnnouncement(SlackMessageSurface):
@@ -696,7 +750,7 @@ class SlackMessagePostMortemReminder5Days(SlackMessageSurface):
         if hasattr(self.incident, "postmortem_for"):
             pm_links.append(
                 ButtonElement(
-                    text="Open Confluence Post-Mortem",
+                    text="Open Post-Mortem (Confluence)",
                     url=self.incident.postmortem_for.page_edit_url,
                     action_id="open_link",
                 )

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_incident-0.0.27.dist-info → firefighter_incident-0.0.28.dist-info}/METADATA

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: firefighter-incident
-Version: 0.0.27
+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.27.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=vq5GihUa4KrDz-Hix5Er9gtVMhfHNDBoLhzP1VsvNuE,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
@@ -291,7 +291,7 @@ firefighter/jira_app/signals/incident_key_events_updated.py,sha256=uaV3MON1QzeOZ
 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
@@ -364,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=eCJUPSjzF_itXH9Z0VlfhloMNvjN3tsB1lmziO9Ksg0,41469
+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
@@ -423,7 +423,7 @@ firefighter/slack/views/modals/downgrade_workflow.py,sha256=cRWsm3DmKRRI1-Jpjprb
 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
@@ -521,12 +521,13 @@ firefighter_tests/test_slack/views/modals/test_form_utils_multiple_choice.py,sha
 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=vbHGx6dkM_0swE1vJ0HrkhI1oJzD_WHZuIQ-_arAxXo,55686
 firefighter_tests/test_slack/views/modals/test_utils.py,sha256=DJd2n9q6fFu8UuCRdiq9U_Cn19MdnC5c-ydLLrk6rkc,5218
-firefighter_incident-0.0.27.dist-info/METADATA,sha256=lHCUnR9PYiRLl_zG_MfvG23UsE_FZWbgQ4zlRdeIilY,5570
-firefighter_incident-0.0.27.dist-info/WHEEL,sha256=WLgqFyCfm_KASv4WHyYy0P3pM_m7J5L9k2skdKLirC8,87
-firefighter_incident-0.0.27.dist-info/entry_points.txt,sha256=c13meJbv7YNmYz7MipMOQwzQ5IeFOPXUBYAJ44XMQsM,61
-firefighter_incident-0.0.27.dist-info/licenses/LICENSE,sha256=krRiGp-a9-1nH1bWpBEdxyTKLhjLmn6DMVVoIb0zF90,1087
-firefighter_incident-0.0.27.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_slack/views/modals/test_postmortem_modal.py

@@ -0,0 +1,72 @@
+from __future__ import annotations
+
+from types import SimpleNamespace
+
+from firefighter.slack.views.modals.postmortem import PostMortemModal
+
+
+class TestPostMortemModal:
+    @staticmethod
+    def _get_text_blocks(view_dict: dict) -> list[str]:
+        return [
+            block.get("text", {}).get("text", "")
+            for block in view_dict.get("blocks", [])
+            if block.get("type") == "section"
+        ]
+
+    @staticmethod
+    def _get_action_elements(view_dict: dict) -> list[dict]:
+        elements: list[dict] = []
+        for block in view_dict.get("blocks", []):
+            if block.get("type") == "actions":
+                elements.extend(block.get("elements", []))
+        return elements
+
+    @staticmethod
+    def test_p1_p2_shows_auto_creation_message(mocker):
+        incident = SimpleNamespace(id=123, needs_postmortem=True)
+
+        # No existing PMs
+        mocker.patch(
+            "firefighter.slack.views.modals.postmortem._safe_has_relation",
+            return_value=False,
+            create=True,
+        )
+
+        view = PostMortemModal().build_modal_fn(incident)
+        view_dict = view.to_dict()
+
+        texts = TestPostMortemModal._get_text_blocks(view_dict)
+        assert any(
+            "automatically created when the incident reaches MITIGATED" in t
+            for t in texts
+        )
+        # No manual-create button for mandatory PMs
+        action_ids = [
+            el.get("action_id")
+            for el in TestPostMortemModal._get_action_elements(view_dict)
+        ]
+        assert "incident_create_postmortem_now" not in action_ids
+
+    @staticmethod
+    def test_p3_shows_optional_message_and_button(mocker):
+        incident = SimpleNamespace(id=456, needs_postmortem=False)
+
+        # No existing PMs
+        mocker.patch(
+            "firefighter.slack.views.modals.postmortem._safe_has_relation",
+            return_value=False,
+            create=True,
+        )
+
+        view = PostMortemModal().build_modal_fn(incident)
+        view_dict = view.to_dict()
+
+        texts = TestPostMortemModal._get_text_blocks(view_dict)
+        assert any("P3 incident post-mortem is not mandatory" in t for t in texts)
+
+        action_ids = [
+            el.get("action_id")
+            for el in TestPostMortemModal._get_action_elements(view_dict)
+        ]
+        assert "incident_create_postmortem_now" in action_ids