From e46632d67a8d7f9af8ae26dfb29dd767142eb59f Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?=D0=A2=D0=B8=D0=BC=D1=83=D1=80=20=D0=93=D0=BE=D1=80=D0=BE?=
 =?UTF-8?q?=D0=B4=D0=B5=D1=86=D0=BA=D0=B8=D0=B9?=
 <gorodetskiy.timur@tnsu.kz>
Date: Fri, 9 Oct 2026 15:44:17 +0500
Subject: [PATCH] =?UTF-8?q?fix:=20=D0=BF=D1=80=D0=B8=D0=BC=D0=B5=D1=87?=
 =?UTF-8?q?=D0=B0=D0=BD=D0=B8=D0=B5=20=D0=A1=D0=91=20=D0=B2=20=D0=B7=D0=B0?=
 =?UTF-8?q?=D1=8F=D0=B2=D0=BA=D0=B5=20=D0=BD=D0=B0=20=D0=BF=D1=80=D0=B8?=
 =?UTF-8?q?=D1=91=D0=BC=20=E2=80=94=20=D0=BD=D0=B5=D0=BE=D0=B1=D1=8F=D0=B7?=
 =?UTF-8?q?=D0=B0=D1=82=D0=B5=D0=BB=D1=8C=D0=BD=D0=BE=D0=B5?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

По просьбе заказчика примечание при отметке «Согласован СБ» больше не
требуется для отправки: поле по-прежнему появляется только при отметке,
но его можно оставить пустым.

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
---
 .../requestPortal/forms/RecruitingHire/model.ts        |  2 +-
 .../forms/RecruitingHire/sections/SecuritySection.tsx  |  4 ++--
 .../forms/RecruitingHire/validation.test.ts            |  9 ++-------
 .../requestPortal/forms/RecruitingHire/validation.ts   | 10 ++--------
 4 files changed, 7 insertions(+), 18 deletions(-)

diff --git a/src/webparts/requestPortal/forms/RecruitingHire/model.ts b/src/webparts/requestPortal/forms/RecruitingHire/model.ts
index a1b3e6e..c901068 100644
--- a/src/webparts/requestPortal/forms/RecruitingHire/model.ts
+++ b/src/webparts/requestPortal/forms/RecruitingHire/model.ts
@@ -86,7 +86,7 @@ export interface IFormData {
   AgencyName: string;
   AgencyTerms: string;
   // Служба безопасности: инициатор при подаче отмечает, что кандидат согласован
-  // с СБ; при отметке обязательно примечание (см. validation.ts). Информационное —
+  // с СБ; примечание при отметке — по желанию. Информационное —
   // на маршрут и решения не влияет.
   HireSecurityApproved: boolean;
   HireSecurityNote: string;
diff --git a/src/webparts/requestPortal/forms/RecruitingHire/sections/SecuritySection.tsx b/src/webparts/requestPortal/forms/RecruitingHire/sections/SecuritySection.tsx
index 4d7d4b5..a8652b8 100644
--- a/src/webparts/requestPortal/forms/RecruitingHire/sections/SecuritySection.tsx
+++ b/src/webparts/requestPortal/forms/RecruitingHire/sections/SecuritySection.tsx
@@ -4,7 +4,7 @@ import { ISectionProps } from '../model';
 
 /**
  * 6. Служба безопасности: инициатор отмечает, что кандидат согласован с СБ, и
- * тогда обязательно пишет примечание (см. validation.ts). Поле информационное —
+ * при желании пишет примечание (необязательно). Поле информационное —
  * на маршрут и решения согласующих не влияет. При снятии отметки примечание
  * очищается, чтобы в заявку не ушёл текст скрытого поля (как в AgencySection).
  */
@@ -20,7 +20,7 @@ export const SecuritySection: React.FC<ISectionProps> = ({ data, set }) => {
         <Toggle checked={data.HireSecurityApproved} onChange={handleToggle} onLabel="Да" />
       </FormField>
       {data.HireSecurityApproved && (
-        <FormField label="6.2 Примечание СБ" htmlFor="HireSecurityNote" required full>
+        <FormField label="6.2 Примечание СБ" htmlFor="HireSecurityNote" full>
           <TextArea
             id="HireSecurityNote"
             value={data.HireSecurityNote}
diff --git a/src/webparts/requestPortal/forms/RecruitingHire/validation.test.ts b/src/webparts/requestPortal/forms/RecruitingHire/validation.test.ts
index ed0198b..032f109 100644
--- a/src/webparts/requestPortal/forms/RecruitingHire/validation.test.ts
+++ b/src/webparts/requestPortal/forms/RecruitingHire/validation.test.ts
@@ -38,13 +38,8 @@ describe('RecruitingHire validation', () => {
     });
   });
 
-  it('«Согласован СБ» требует примечание СБ, без отметки — не требует', () => {
-    expect(labels(submitIssues(DRAFT_READY))).not.toContain('6.2 Примечание СБ');
-    expect(labels(submitIssues({ ...DRAFT_READY, HireSecurityApproved: true }))).toContain('6.2 Примечание СБ');
-    expect(
-      labels(submitIssues({ ...DRAFT_READY, HireSecurityApproved: true, HireSecurityNote: 'Проверка пройдена' }))
-    ).not.toContain('6.2 Примечание СБ');
-    // Черновик без примечания сохранить можно.
+  it('примечание СБ необязательно, даже при отметке «Согласован СБ»', () => {
+    expect(labels(submitIssues({ ...DRAFT_READY, HireSecurityApproved: true }))).not.toContain('6.2 Примечание СБ');
     expect(draftIssues({ ...DRAFT_READY, HireSecurityApproved: true })).toEqual([]);
   });
 
diff --git a/src/webparts/requestPortal/forms/RecruitingHire/validation.ts b/src/webparts/requestPortal/forms/RecruitingHire/validation.ts
index 3f82dd6..29ec007 100644
--- a/src/webparts/requestPortal/forms/RecruitingHire/validation.ts
+++ b/src/webparts/requestPortal/forms/RecruitingHire/validation.ts
@@ -19,7 +19,7 @@ interface IRequiredRule extends IFieldIssue {
  * агентство» не проверяется — у булевых нет пустого состояния.
  * 2.2 «Дата приёма» не нужна, если отмечено «По приезду на объект»
  * (HireOnArrival) — тогда дату проставляет исполнитель на своём шаге.
- * 6.2 «Примечание СБ» обязательно только при отметке 6.1 «Согласован СБ».
+ * 6.2 «Примечание СБ» необязательно (по просьбе заказчика) — даже при отметке 6.1.
  *
  * 1.3 «Проект» — хотя бы один проект из справочника: первый уходит в resolve
  * маршрута (projectId), а без него approval-route отвечал 404 и заявка ушла бы
@@ -49,13 +49,7 @@ const REQUIRED_RULES: ReadonlyArray<IRequiredRule> = [
   { field: 'ProbationPeriod', label: '4.1 Испытательный срок', ok: (d) => !!d.ProbationPeriod },
   { field: 'SalaryProbation', label: '4.2 Оклад при испытательном сроке', ok: (d) => !!d.SalaryProbation.trim() },
   { field: 'SalaryAfterProbation', label: '4.3 Оклад по завершении ИС', ok: (d) => !!d.SalaryAfterProbation.trim() },
-  { field: 'WorkSchedule', label: '4.4 График работы', ok: (d) => !!d.WorkSchedule },
-  {
-    field: 'HireSecurityNote',
-    label: '6.2 Примечание СБ',
-    message: 'отмечено «Согласован СБ» — укажите примечание',
-    ok: (d) => !d.HireSecurityApproved || !!d.HireSecurityNote.trim()
-  }
+  { field: 'WorkSchedule', label: '4.4 График работы', ok: (d) => !!d.WorkSchedule }
 ];
 
 function toIssue({ field, label, message }: IRequiredRule): IFieldIssue {