From a178fcd3fbf8fa2dda08724f36ac6b1dbf88777f 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, 29 Dec 2025 11:26:45 +0300 Subject: [PATCH] =?UTF-8?q?feat(requests):=20=D1=83=D0=BB=D1=83=D1=87?= =?UTF-8?q?=D1=88=D0=B5=D0=BD=D1=8B=20=D1=83=D0=B2=D0=B5=D0=B4=D0=BE=D0=BC?= =?UTF-8?q?=D0=BB=D0=B5=D0=BD=D0=B8=D1=8F=20=D0=BE=20=D0=B7=D0=B0=D1=8F?= =?UTF-8?q?=D0=B2=D0=BB=D0=B5=D0=BD=D0=B8=D1=8F=D1=85?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit - Исправлена терминология: 'заявка' → 'заявление' во всех уведомлениях - Добавлены эмодзи-иконки для визуального различия типов уведомлений - Разделены типы уведомлений для основных получателей и cc_users: * request_new_primary - � адресовано непосредственно * request_new_cc - � в копии (информирование) * request_new_approver - ✅ на согласование * request_new_dept - � в отделе - Добавлены детали в уведомления: * Имя автора заявления * Тип заявления * Превью комментария (до 150 символов) * Имя approver в уведомлениях о решении - Улучшены заголовки с указанием типа заявления - В metadata добавлены employee_name и approver_name --- backend/requests_app/notification_signals.py | 110 ++++++++++--------- 1 file changed, 60 insertions(+), 50 deletions(-) diff --git a/backend/requests_app/notification_signals.py b/backend/requests_app/notification_signals.py index 958ea830..60092c45 100644 --- a/backend/requests_app/notification_signals.py +++ b/backend/requests_app/notification_signals.py @@ -2,11 +2,11 @@ Signals для автоматической генерации уведомлений в модуле Requests. Обрабатывает события: -- Новая заявка (для ответственных/руководителей) -- Одобрение заявки -- Отклонение заявки -- Комментарий к заявке -- Изменение статуса заявки +- Новое заявление (для ответственных/руководителей) +- Одобрение заявления +- Отклонение заявления +- Комментарий к заявлению +- Изменение статуса заявления """ from django.contrib.auth import get_user_model @@ -22,10 +22,10 @@ Employee = get_user_model() @receiver(post_save, sender=Request) def create_request_notifications(sender, instance, created, **kwargs): """ - Создает уведомления при создании или изменении заявки. + Создает уведомления при создании или изменении заявления. Обрабатывает: - 1. Новая заявка - уведомление руководителю/ответственным + 1. Новое заявление - уведомление руководителю/ответственным 2. Изменение статуса - уведомление автору """ request_obj = instance @@ -60,9 +60,9 @@ def track_status_change(sender, instance, **kwargs): @receiver(post_save, sender=RequestComment) def create_comment_notification(sender, instance, created, **kwargs): """ - Создает уведомление при добавлении комментария к заявке. + Создает уведомление при добавлении комментария к заявлению. Уведомляет: - - Автора заявки + - Автора заявления - Всех получателей - Всех в копии - Согласующего @@ -109,14 +109,18 @@ def create_comment_notification(sender, instance, created, **kwargs): recipients_set.update(dept_employees) # Отправляем уведомления + author_name = author.get_full_name() or author.username + request_type = request_obj.get_type_display() + employee_name = request_obj.employee.get_full_name() or request_obj.employee.username + for recipient in recipients_set: NotificationService.create_notification( recipient=recipient, notification_type_code="request_comment", - title="Новый комментарий к заявке", + title=f"💬 Новый комментарий к заявлению от {employee_name}", message=( - f"{author.get_full_name() or author.username} " - f"прокомментировал заявку: {comment.text[:100]}" + f"{author_name} прокомментировал заявление " + f'"{request_type}": {comment.text[:100]}' ), content_object=request_obj, action_url=f"/requests/{request_obj.id}/", @@ -134,9 +138,9 @@ def create_comment_notification(sender, instance, created, **kwargs): def notify_new_request(request_obj): """ - Отправляет уведомление о новой заявке: - - Всем основным получателям (recipients) - - Всем в копии (cc_users) + Отправляет уведомление о новом заявлении: + - Всем основным получателям (recipients) - адресованное им + - Всем в копии (cc_users) - с пометкой "в копии" - Согласующему (approver) - Руководителям отделов - Пользователям с правом can_process_requests @@ -202,6 +206,8 @@ def notify_new_request(request_obj): # Определяем тип уведомления для каждого получателя author_name = request_obj.employee.get_full_name() or request_obj.employee.username + request_type = request_obj.get_type_display() + comment_preview = request_obj.comment[:150] if request_obj.comment else "Без комментария" for recipient in recipients_set: # Определяем роль получателя @@ -209,24 +215,26 @@ def notify_new_request(request_obj): is_cc = request_obj.cc_users.filter(id=recipient.id).exists() is_approver = request_obj.approver_id == recipient.id - # Формируем заголовок и сообщение - if is_approver: - title = f"Новая заявка на согласование от {author_name}" - notification_type = "request_new" - elif is_primary: - title = f"Вам адресована заявка от {author_name}" - notification_type = "request_new" + # Формируем заголовок и сообщение с учетом роли + if is_primary: + # Основной получатель - требует действия + title = f"📩 Вам адресовано заявление от {author_name}" + message = f'Тип: "{request_type}". {comment_preview}' + notification_type = "request_new_primary" elif is_cc: - title = f"Вы в копии заявки от {author_name}" - notification_type = "request_new" + # Копия - информирование + title = f"📋 Вы в копии заявления от {author_name}" + message = f'Тип: "{request_type}". {comment_preview}' + notification_type = "request_new_cc" + elif is_approver: + title = f"✅ Новое заявление на согласование от {author_name}" + message = f'Тип: "{request_type}". {comment_preview}' + notification_type = "request_new_approver" else: - title = f"Новая заявка в отделе от {author_name}" - notification_type = "request_new" - - message = ( - f"Тип: {request_obj.get_type_display()}. " - f"{request_obj.comment[:100] if request_obj.comment else ''}" - ) + # Руководитель отдела или обработчик + title = f"📝 Новое заявление в отделе от {author_name}" + message = f'Тип: "{request_type}". {comment_preview}' + notification_type = "request_new_dept" NotificationService.create_notification( recipient=recipient, @@ -239,6 +247,7 @@ def notify_new_request(request_obj): "request_id": request_obj.id, "request_type": request_obj.type, "employee_id": request_obj.employee.id, + "employee_name": author_name, "is_primary_recipient": is_primary, "is_cc": is_cc, "is_approver": is_approver, @@ -300,38 +309,36 @@ def notify_status_change(request_obj, old_status, new_status): recipients_to_notify.update(dept_employees) - # Формируем уведомления + # Формируем уведомления с подробной информацией + request_type = request_obj.get_type_display() + employee_name = request_obj.employee.get_full_name() or request_obj.employee.username + approver_name = ( + request_obj.approver.get_full_name() + if request_obj.approver + else "Руководитель" + ) + for recipient in recipients_to_notify: if new_status == "approved": notification_type = "request_approved" - title = "Заявка одобрена" - approver_name = ( - request_obj.approver.get_full_name() - if request_obj.approver - else "Руководитель" - ) + title = f"✅ Заявление одобрено: {request_type}" message = ( - f'Заявка "{request_obj.get_type_display()}" ' - f"одобрена пользователем {approver_name}" + f'Заявление от {employee_name} "{request_type}" ' + f"одобрено пользователем {approver_name}" ) elif new_status == "rejected": notification_type = "request_rejected" - title = "Заявка отклонена" - approver_name = ( - request_obj.approver.get_full_name() - if request_obj.approver - else "Руководитель" - ) + title = f"❌ Заявление отклонено: {request_type}" message = ( - f'Заявка "{request_obj.get_type_display()}" ' - f"отклонена пользователем {approver_name}" + f'Заявление от {employee_name} "{request_type}" ' + f"отклонено пользователем {approver_name}" ) else: # Общее уведомление об изменении статуса notification_type = "request_status_changed" - title = "Статус заявки изменен" + title = f"🔄 Статус заявления изменен: {request_type}" message = ( - f'Статус заявки "{request_obj.get_type_display()}" ' + f'Статус заявления от {employee_name} "{request_type}" ' f"изменен: {old_status} → {new_status}" ) @@ -345,10 +352,13 @@ def notify_status_change(request_obj, old_status, new_status): metadata={ "request_id": request_obj.id, "request_type": request_obj.type, + "employee_id": request_obj.employee.id, + "employee_name": employee_name, "old_status": old_status, "new_status": new_status, "approver_id": ( request_obj.approver.id if request_obj.approver else None ), + "approver_name": approver_name, }, )