aurora-1799 отображение ошибки в случае выбора только дт или кт в таблице #207

- режим запроса вынесен в utils/chessMatrixMode (all / manual / half) с тестами;
  при «одна сторона» запрос матрицы не уходит, индикатор загрузки снимается;
- подсказка по спеке intercompany-matrix-ui: один текст в таблице и у фильтров,
  у фильтров — только в состоянии «одна сторона»;
- высота шахматки: min-height только для состояния подсказки, обёртка снова auto
  (в обычном режиме «Итого» сразу под строками);
- откат pairs?.length в shouldWarnCounterPairs (маскировал старый API);
- вернули комментарий про сумму, лишний columns || [] и дубль CSS убраны;
  опечатка CataloguGroupTabsI.

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>

aep commented

2026-10-29 20:39:03 +03:00

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

Прошёл сценарий на стенде и поправил сам, коммит d3c4d35 в ветке:

При выборе только Дт или только Кт уходил лишний GET …/intercompany-matrix/all. Теперь режим считает utils/chessMatrixMode (all / manual / одна сторона), в одной стороне запрос не шлём, спиннер снимается. Покрыл 4 случая в vitest.
pairs?.length в shouldWarnCounterPairs откатил: counter_pairs бэк отдаёт всегда, undefined бывает только на старом API, а с ?. вылезала пустая модалка «0 пар у 0 участников».
Высота: min-height 550px только для состояния подсказки, у обёртки снова auto. В обычном режиме «Итого» сразу под строками, как в макете.
Текст подсказки по спеке «Для расчёта по счетам выберите счета и в Дт, и в Кт» — один в таблице и у фильтров, у фильтров только когда выбрана одна сторона.
Мелочи: вернул комментарий про «не выдумываем нулём», убрал columns || [] и дубль в CSS, поправил CataloguGroupTabsI.

Проверено: tsc, eslint, prettier, vitest. Руками: только Дт, оба (Дт 62 / Кт 60 — цифры сходятся с эталоном), сброс. Полноэкранный режим не смотрел. Переименование каталогов ок, в следующий раз лучше отдельным PR.

Ревью, раунд 1 Прошёл сценарий на стенде и поправил сам, коммит d3c4d35 в ветке: 1. При выборе только Дт или только Кт уходил лишний GET …/intercompany-matrix/all. Теперь режим считает utils/chessMatrixMode (all / manual / одна сторона), в одной стороне запрос не шлём, спиннер снимается. Покрыл 4 случая в vitest. 2. pairs?.length в shouldWarnCounterPairs откатил: counter_pairs бэк отдаёт всегда, undefined бывает только на старом API, а с ?. вылезала пустая модалка «0 пар у 0 участников». 3. Высота: min-height 550px только для состояния подсказки, у обёртки снова auto. В обычном режиме «Итого» сразу под строками, как в макете. 4. Текст подсказки по спеке «Для расчёта по счетам выберите счета и в Дт, и в Кт» — один в таблице и у фильтров, у фильтров только когда выбрана одна сторона. 5. Мелочи: вернул комментарий про «не выдумываем нулём», убрал columns || [] и дубль в CSS, поправил CataloguGroupTabsI. Проверено: tsc, eslint, prettier, vitest. Руками: только Дт, оба (Дт 62 / Кт 60 — цифры сходятся с эталоном), сброс. Полноэкранный режим не смотрел. Переименование каталогов ок, в следующий раз лучше отдельным PR.

aep merged commit da5cb9c222 into main

2026-10-29 20:40:33 +03:00

aep referenced this issue from a commit

2026-10-29 20:40:37 +03:00

Merge pull request 'aurora-1799 отображение ошибки в случае выбора только дт или кт в таблице' (#207) from fix/chessTable into main

aep referenced this pull request

2026-10-29 20:41:39 +03:00

Follow-ups from PR #207 (шахматка, AURORA-1799) #209

Sign in to join this conversation.

2 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: RSHB/arm-core#207