RSHB/arm-core
9
0
Fork 0
Code Issues 24 Pull Requests 4 Actions Packages Projects Releases Wiki Activity

AURORA-1671: исправления по функционалу ГК и ЮЛ #179

Merged
aep merged 4 commits from fix/legalEntities into main 2026-10-26 23:17:18 +03:00
Conversation 1 Commits 4 Files Changed 20 +160 -174
grog commented 2026-10-26 13:47:07 +03:00
Owner
Copy Link
  • исправлен баг формы редактирования ЮЛ, с получением ГСК и ГСЗ
  • исправлен баг формы ГСК/ГСЗ c отправкой сегодняшней даты при редактировании (сейчас только при создании),
  • выведена ошибка INN_EXISTS в форму ЮЛ напрямую
  • отправка пустых значений в виде null вместо undefined
  • Список ЮЛ — все страницы: сейчас читаются первые 200 записей; дочитывать страницы до total (оставлен, в будующем исправить на бэк).
  • автоматическое добавление активного ГК в формы ГСК/ГСЗ и ЮЛ
  • исправлен баг с фильтром ГСК/ГСЗ
  • добавлена колонка crm id в таблицы ГСК/ГСЗ
  • выведена ошибка CRM_ID_EXISTS в форму ГСЗ/ГСК напрямую
  • убрана подсказка "Генерируется автоматически" в формах у crm_id
- исправлен баг формы редактирования ЮЛ, с получением ГСК и ГСЗ - исправлен баг формы ГСК/ГСЗ c отправкой сегодняшней даты при редактировании (сейчас только при создании), - выведена ошибка INN_EXISTS в форму ЮЛ напрямую - отправка пустых значений в виде null вместо undefined - Список ЮЛ — все страницы: сейчас читаются первые 200 записей; дочитывать страницы до total (оставлен, в будующем исправить на бэк). - автоматическое добавление активного ГК в формы ГСК/ГСЗ и ЮЛ - исправлен баг с фильтром ГСК/ГСЗ - добавлена колонка crm id в таблицы ГСК/ГСЗ - выведена ошибка CRM_ID_EXISTS в форму ГСЗ/ГСК напрямую - убрана подсказка "Генерируется автоматически" в формах у crm_id
grog added 1 commit 2026-10-26 13:47:09 +03:00
grog requested review from aep 2026-10-26 13:47:09 +03:00
grog requested review from maxim 2026-10-26 13:47:11 +03:00
grog added 1 commit 2026-10-26 14:12:03 +03:00
grog added 1 commit 2026-10-26 14:15:11 +03:00
aep added 1 commit 2026-10-26 23:00:59 +03:00 
- Страница ГК: снова выбирается первая ГК списка; список ЮЛ без выбранной ГК
  не запрашивается (иначе уходил запрос по всем ГК), «ЮЛ вне ГК» грузится без ГК.
- Выбранная ГК обновляется после перезагрузки списка ГК — счётчик ЮЛ в шапке
  и на вкладке после создания/переноса/вывода ЮЛ.
- Загрузка ГСК/ГСЗ: try/finally — ошибка не оставляет вечный спиннер.
- Форма ЮЛ: INVALID_INN/INVALID_KPP/INN_EXISTS/CRM_ID_EXISTS — у поля;
  КПП приводится к верхнему регистру при вводе.
- crm_id ГСК/ГСЗ — string | null; колонка «ID в CRM»; LE_PAGE вместо литерала;
  возвращён комментарий в фильтре шахматки.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
aep commented 2026-10-26 23:08:24 +03:00
Owner
Copy Link

Ревью, раунд 1

По задаче почти всё закрыто. Несколько мест поправил сам коммитом 3aad670, чтобы не гонять лишний круг:

  1. Страница ГК без выбранной ГК. Вместе с переносом activeCompanyGroup в catalogue_jur пропал автовыбор первой ГК, а guard в CatalogueJurTable снят, поэтому таблица грузила ЮЛ без фильтра по ГК. Автовыбор вернул, guard теперь outside_group || activeCompanyGroup?.id. «ЮЛ вне ГК» после F5 грузится, это закрывает AURORA-1688.
  2. Счётчик ЮЛ в шапке и на вкладке не обновлялся после создания, переноса или вывода ЮЛ: при переезде потерялось обновление выбранной ГК после getCompanyGroupsThunk. Добавил extraReducer в catalogue_jur.
  3. В getRelatedGroupsHandler без try/finally при ошибке крутился спиннер, вернул.
  4. В форме ЮЛ у поля теперь и INVALID_INN / INVALID_KPP (сделал таблицей «код → поле»). КПП приводится к верхнему регистру через normalize, иначе строчные отбивал наш же regex.
  5. Мелочи: crm_id у ГСК/ГСЗ string | null, колонка «ID в CRM» как в форме, вернул LE_PAGE и комментарий в фильтре шахматки.

Пустые crm_id и КПП бэк сам приводит к null, тут ничего не нужно. Пагинацию списка ЮЛ оставляем техдолгом.

На будущее: try/catch из обработчиков убирать можно, когда ошибку ловит форма. Но где есть loading, finally оставляем.

Проверил на стенде, web-проверки зелёные. С моей стороны можно сливать.

**Ревью, раунд 1** По задаче почти всё закрыто. Несколько мест поправил сам коммитом 3aad670, чтобы не гонять лишний круг: 1. Страница ГК без выбранной ГК. Вместе с переносом activeCompanyGroup в catalogue_jur пропал автовыбор первой ГК, а guard в CatalogueJurTable снят, поэтому таблица грузила ЮЛ без фильтра по ГК. Автовыбор вернул, guard теперь `outside_group || activeCompanyGroup?.id`. «ЮЛ вне ГК» после F5 грузится, это закрывает AURORA-1688. 2. Счётчик ЮЛ в шапке и на вкладке не обновлялся после создания, переноса или вывода ЮЛ: при переезде потерялось обновление выбранной ГК после getCompanyGroupsThunk. Добавил extraReducer в catalogue_jur. 3. В getRelatedGroupsHandler без try/finally при ошибке крутился спиннер, вернул. 4. В форме ЮЛ у поля теперь и INVALID_INN / INVALID_KPP (сделал таблицей «код → поле»). КПП приводится к верхнему регистру через normalize, иначе строчные отбивал наш же regex. 5. Мелочи: crm_id у ГСК/ГСЗ `string | null`, колонка «ID в CRM» как в форме, вернул LE_PAGE и комментарий в фильтре шахматки. Пустые crm_id и КПП бэк сам приводит к null, тут ничего не нужно. Пагинацию списка ЮЛ оставляем техдолгом. На будущее: try/catch из обработчиков убирать можно, когда ошибку ловит форма. Но где есть loading, finally оставляем. Проверил на стенде, web-проверки зелёные. С моей стороны можно сливать.
aep merged commit ed0052bad0 into main 2026-10-26 23:17:18 +03:00
Sign in to join this conversation.
Reviewers
No Reviewers
aep
maxim
Labels
No items
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
PotapovaA Raykov-MS SalminMP Veronika Yashin1994 aep grog maxim tsygankoviva
No Assignees
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: RSHB/arm-core#179
No description provided.
Delete Branch "fix/legalEntities"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?