Skip to content

Понятные ошибки настройки и заголовок программы - #515

Merged
TThanos3000 merged 2 commits into
devfrom
fix/dev-program-setup-validation
Oct 30, 2026
Merged

TThanos3000 merged 2 commits into
devfrom
fix/dev-program-setup-validation

Conversation

@TThanos3000

@TThanos3000 TThanos3000 commented Oct 29, 2026 •

Copy link
Copy Markdown
Contributor

Что изменено

Неверные сроки программы показывают ошибки у полей и не отправляются на сервер; введенные значения сохраняются. Неполный шаблон решения направляет к добавлению кейсов, задач или вариантов. Название программы видно на desktop и mobile, а ошибка анкеты вошедшего пользователя не просит отсутствующих согласий.

Зачем

На dev несовместимые даты сохранялись без предупреждения, пустой шаблон давал общую ошибку, а настольная страница и предпросмотр не показывали текстовое название программы.

Реализация

Проверка порядка дат по Москве, aria-invalid и связанная подсказка; неполный черновик и подтверждение повторного открытия сроков сохранены. Начальные шаблоны используют существующий редактор поля. Публичная оболочка решений сохраняет свой заголовок: detail не добавляет второй desktop h1. Регрессия воспроизведена двумя интеграционными тестами public wrapper + detail до исправления и закрыта после него, с обложкой и без нее.

API

Новых методов и полей нет, успешные payloads не менялись. Авторитетная серверная проверка находится в API №845, который был слит и развернут до Angular.

UI

Существующие Button, Alert, редактор поля, Mont и layout сохранены. Desktop 1280 и mobile 390 проверены; один видимый заголовок, без горизонтального overflow. Внешние изображения и шрифты блокировались в изолированном browser harness.

Permissions

Не менялись. Используется прежний management context; frontend validation не заменяет server-side проверки.

Проверки

Окончательный head: a0b5b4aae7753a9281d3fb7da00b96da013bc884. Merge в dev: 693dc41f5e7b394d701c9835a65342fd220823d0.

  • Checked by CI на точном merge: Angular CI/deploy 41066505993 success: npm ci, полный format:check, lint:ts, 497 файлов / 3075 тестов, solution-contract types, build:pr и rsync.
  • Checked locally на окончательном head: 83 теста четырех затронутых наборов, solution-contract, повторная development build, Prettier двух финальных файлов, ESLint измененного теста и diff check PASS.
  • Checked in browser, локальная связка: реальный Angular → API №845 → отдельная PostgreSQL база. Неверные даты не отправляются и не теряют ввод; 409 до подтверждения; сохранение регистрации до конца программы включительно; partial draft. Пустой шаблон открывает редактор кейса без неудачного save и сохраняется после добавления варианта. Авторизованная анкета показывает правильную ошибку.
  • Checked in browser, опубликованный dev: публичная страница решений на 1280/390, один видимый заголовок, Mont, без горизонтального overflow и ошибок JavaScript. Записывающие запросы блокировались. Публичный просмотр выполнялся без аккаунта.
  • Checked by HTTP: обновленный SPA bundle, admin/login, публичные решения, закрытые API 401 и безопасные невалидные запросы авторизации 400. Штатная проверка SPA/API маршрутов PASS.
  • Requirement audit и source review затронутых flows завершены, незакрытых замечаний нет.

Acceptance criteria

  • Окончание позже начала, регистрация не позже окончания и может идти после старта — tests, local browser и API readback.
  • Неполный календарь черновика сохраняется — tests и local browser.
  • Пустой список кейсов исправляется прямым действием — tests и реальное сохранение в local browser.
  • Название видно с обложкой и без нее — component/integration tests; один видимый h1 на desktop/mobile опубликованного dev.
  • Ошибка регистрации соответствует текущему пользователю — component tests и authenticated local browser.

Не проверено

Авторизованное редактирование сроков на dev после merge: в облачной сессии нет входа в dev. Этот flow проверен локально с настоящими Angular/API/PostgreSQL. Из базового Vitest config по-прежнему исключены projects.component.spec.ts, program.component.spec.ts и editor-submit-button.directive.spec.ts; они не входят в 497 файлов. Production и внешние интеграции не проверялись.

Риски / migration

Миграции не нужны. API №845 → успешный dev deploy → Angular №515 выполнены по запросу пользователя. API deploy и Angular deploy success. Упрощение навигации первичной настройки остается отдельной задачей. Прод не затрагивался.

@TThanos3000
TThanos3000 merged commit 693dc41 into dev Oct 30, 2026
@TThanos3000
TThanos3000 deleted the fix/dev-program-setup-validation branch October 30, 2026 13:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant