From aa97657721c15236b48b20f27559b4a8d435d277 Mon Sep 17 00:00:00 2001
From: "gorbenko.pavel" <gorbenko.pavel@tansu.kz>
Date: Thu, 20 Aug 2026 11:49:54 +0500
Subject: [PATCH] =?UTF-8?q?=D0=BE=D0=B1=D0=BD=D0=BE=D0=B2=D0=BB=D0=B5?=
 =?UTF-8?q?=D0=BD=20reminders?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 .../Reminders/ReminderService.cs                     |  9 +++++++--
 .../Reminders/RemindersOptions.cs                    | 12 +++++++++---
 .../ApplicationServices/ReminderServiceTests.cs      |  7 ++++---
 3 files changed, 20 insertions(+), 8 deletions(-)

diff --git a/backend/Tnsu.ApprovalRoute.Application/Reminders/ReminderService.cs b/backend/Tnsu.ApprovalRoute.Application/Reminders/ReminderService.cs
index 0d29747..6808b87 100644
--- a/backend/Tnsu.ApprovalRoute.Application/Reminders/ReminderService.cs
+++ b/backend/Tnsu.ApprovalRoute.Application/Reminders/ReminderService.cs
@@ -191,7 +191,7 @@ public sealed class ReminderService(
             reminders.Add(new ReminderDto(
                 row.ApproverEmail!,
                 (int)Math.Floor((now - deadline).TotalHours),
-                BuildUrl(type, row),
+                BuildUrl(type, row, listId),
                 row.AskOverdueReason ?? type.AskOverdueReason ?? false,
                 type.Id,
                 type.Name,
@@ -207,12 +207,17 @@ public sealed class ReminderService(
     /// Ссылка ведёт в тот раздел портала, которому принадлежит просроченный шаг:
     /// исполнителя незачем высаживать в «На согласовании». Сама заявка всё равно
     /// откроется панелью поверх раздела (rpmode=approve).
+    ///
+    /// listId — server-relative url партиции (см. shared/routing.ts в
+    /// request-portal, параметр rplist): без него архивная заявка открылась бы
+    /// так, будто лежит в ТЕКУЩЕЙ партиции типа, и портал не нашёл бы её.
     /// </summary>
-    private string BuildUrl(RequestType type, ApprovalRow row) =>
+    private string BuildUrl(RequestType type, ApprovalRow row, string listId) =>
         options.Value.RequestUrlTemplate
             .Replace("{siteUrl}", type.SiteUrl!.TrimEnd('/'))
             .Replace("{pageUrl}", type.PageUrl)
             .Replace("{requestId}", row.TicketId.ToString())
+            .Replace("{listId}", Uri.EscapeDataString(listId))
             .Replace("{nav}", row.Phase == ApprovalPhases.Execution ? "execution" : "approvals");
 
     /// <summary>«Заявка на приём — данные» → «Заявка на приём — согласование».</summary>
diff --git a/backend/Tnsu.ApprovalRoute.Application/Reminders/RemindersOptions.cs b/backend/Tnsu.ApprovalRoute.Application/Reminders/RemindersOptions.cs
index 54ff0ac..2776b5e 100644
--- a/backend/Tnsu.ApprovalRoute.Application/Reminders/RemindersOptions.cs
+++ b/backend/Tnsu.ApprovalRoute.Application/Reminders/RemindersOptions.cs
@@ -3,19 +3,25 @@ namespace Tnsu.ApprovalRoute.Application.Reminders;
 public sealed class RemindersOptions
 {
     /// <summary>
-    /// Шаблон ссылки на заявку. Подстановки: {siteUrl}, {pageUrl}, {requestId}, {nav}.
+    /// Шаблон ссылки на заявку. Подстановки: {siteUrl}, {pageUrl}, {requestId},
+    /// {listId}, {nav}.
     ///
     /// Портал — одностраничный и живёт в КОРНЕ сайта, отдельного пути к странице
     /// нет; всё состояние передаётся параметрами запроса (см. shared/routing.ts
     /// в request-portal). Пример живой ссылки:
-    /// https://tnsukz.sharepoint.com/sites/requests?rpnav=approvals&amp;rpmode=approve&amp;rptype=it-1c-ticket&amp;rpid=11
+    /// https://tnsukz.sharepoint.com/sites/requests?rpnav=approvals&amp;rpmode=approve&amp;rptype=it-1c-ticket&amp;rpid=11&amp;rplist=%2Fsites%2Frequests%2FLists%2F...
     ///
     /// {pageUrl} — это PageUrl типа заявки, то есть КЛЮЧ ФОРМЫ (it-1c-ticket), а
     /// не путь к странице: в портале он и подставляется в rptype.
     ///
+    /// {listId} — server-relative url КОНКРЕТНОЙ партиции, где лежит строка
+    /// согласования (параметр rplist). Без него портал открывал бы заявку так,
+    /// будто она лежит в ТЕКУЩЕЙ партиции типа, и архивная заявка не находилась
+    /// бы вовсе (см. IRoute.listId в request-portal/shared/routing.ts).
+    ///
     /// Вынесено в конфигурацию, чтобы смена разметки ссылок в портале правилась
     /// настройкой, а не пересборкой сервиса.
     /// </summary>
     public string RequestUrlTemplate { get; set; } =
-        "{siteUrl}?rpnav={nav}&rpmode=approve&rptype={pageUrl}&rpid={requestId}";
+        "{siteUrl}?rpnav={nav}&rpmode=approve&rptype={pageUrl}&rpid={requestId}&rplist={listId}";
 }
diff --git a/backend/tests/Tnsu.ApprovalRoute.IntegrationTests/ApplicationServices/ReminderServiceTests.cs b/backend/tests/Tnsu.ApprovalRoute.IntegrationTests/ApplicationServices/ReminderServiceTests.cs
index ce1f2bc..778b2e9 100644
--- a/backend/tests/Tnsu.ApprovalRoute.IntegrationTests/ApplicationServices/ReminderServiceTests.cs
+++ b/backend/tests/Tnsu.ApprovalRoute.IntegrationTests/ApplicationServices/ReminderServiceTests.cs
@@ -215,8 +215,8 @@ public sealed class ReminderServiceTests : IClassFixture<PostgresDatabaseFixture
     }
 
     // Портал — одностраничный и живёт в корне сайта; вид ссылки подтверждён
-    // живым примером:
-    // https://tnsukz.sharepoint.com/sites/requests?rpnav=approvals&rpmode=approve&rptype=it-1c-ticket&rpid=11
+    // живым примером (см. docs/routing.md в request-portal):
+    // https://tnsukz.sharepoint.com/sites/requests?rpnav=approvals&rpmode=approve&rptype=it-1c-ticket&rpid=11&rplist=%2Fsites%2Frequests%2FLists%2Fit-1c-ticket
     [Fact(DisplayName = "Ссылка на заявку собирается в том же виде, что и в портале")]
     public async Task Url_MatchesPortalFormat()
     {
@@ -228,7 +228,8 @@ public sealed class ReminderServiceTests : IClassFixture<PostgresDatabaseFixture
 
         result.Value.ShouldHaveSingleItem().Url.ShouldBe(
             "https://tnsukz.sharepoint.com/sites/requests" +
-            "?rpnav=approvals&rpmode=approve&rptype=/it-1c-ticket&rpid=11");
+            "?rpnav=approvals&rpmode=approve&rptype=/it-1c-ticket&rpid=11" +
+            "&rplist=%2Fsites%2Frequests%2FLists%2Fit-1c-ticket");
     }
 
     [Fact(DisplayName = "Просроченный шаг исполнения ведёт в раздел «На исполнении», а не в согласование")]