From d84619f6a08dd97b701ef7789e083024f92e0636 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?=D0=98=D0=B3=D0=BE=D1=80=D1=8C=20=D0=9D=D0=B0=D0=B4=D0=B5?= =?UTF-8?q?=D0=B8=D0=BD?= Date: Mon, 5 Jan 2026 07:11:46 +0300 Subject: [PATCH] =?UTF-8?q?fix:=20=D0=B0=D0=B2=D1=82=D0=BE=D1=80=20=D0=B7?= =?UTF-8?q?=D0=B0=D1=8F=D0=B2=D0=BB=D0=B5=D0=BD=D0=B8=D1=8F=20=D0=B4=D0=BE?= =?UTF-8?q?=D0=BB=D0=B6=D0=B5=D0=BD=20=D0=BF=D0=BE=D0=BB=D1=83=D1=87=D0=B0?= =?UTF-8?q?=D1=82=D1=8C=20=D1=83=D0=B2=D0=B5=D0=B4=D0=BE=D0=BC=D0=BB=D0=B5?= =?UTF-8?q?=D0=BD=D0=B8=D1=8F=20=D0=BE=20=D1=80=D0=B5=D1=88=D0=B5=D0=BD?= =?UTF-8?q?=D0=B8=D0=B8=20(approve/reject)?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit - Удалено исключение employee.id из notify_status_change() - Коммит 087cff79 ошибочно исключал автора, предполагая что он видит результат на странице - После удаления alert() в коммите 1a5de8c уведомление стало единственным способом информирования - Теперь автор получает уведомление о одобрении/отклонении заявления - Approver остается исключенным (он сам нажал кнопку, не нужно ему уведомление) Fixes: Users not receiving notifications when their request is approved/rejected --- backend/requests_app/notification_signals.py | 11 +++++------ 1 file changed, 5 insertions(+), 6 deletions(-) diff --git a/backend/requests_app/notification_signals.py b/backend/requests_app/notification_signals.py index 7f6f6423..e2808695 100644 --- a/backend/requests_app/notification_signals.py +++ b/backend/requests_app/notification_signals.py @@ -355,15 +355,14 @@ def notify_status_change(request_obj, old_status, new_status): # ID пользователей которых НЕ нужно уведомлять при approve/reject exclude_ids = set() if new_status in ("approved", "rejected"): - # Не уведомляем автора заявки (он видит результат на странице) - exclude_ids.add(request_obj.employee.id) - # Не уведомляем того кто принял решение (он сам нажал кнопку) + # ВАЖНО: Уведомляем автора заявки о решении - это единственный способ + # информировать его, так как alert() был убран из UI (коммит 935c12a) + # Не уведомляем только того кто принял решение (он сам нажал кнопку) if request_obj.approver_id: exclude_ids.add(request_obj.approver_id) - # 1. Автор - НЕ уведомляем при approved/rejected - if new_status not in ("approved", "rejected"): - recipients_to_notify.add(request_obj.employee) + # 1. Автор - УВЕДОМЛЯЕМ всегда (даже при approved/rejected, это важно!) + recipients_to_notify.add(request_obj.employee) # 2. Основные получатели (исключая тех кто в exclude_ids) recipients_to_notify.update(