Repository navigation
Понятные ошибки настройки и заголовок программы - #515
Merged
Merged
Conversation
10 tasks done
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Что изменено
Неверные сроки программы показывают ошибки у полей и не отправляются на сервер; введенные значения сохраняются. Неполный шаблон решения направляет к добавлению кейсов, задач или вариантов. Название программы видно на 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.Acceptance criteria
Не проверено
Авторизованное редактирование сроков на 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. Упрощение навигации первичной настройки остается отдельной задачей. Прод не затрагивался.