Repository navigation
Unify expert project scoring and program evaluation lists - #506
Merged
Merged
Conversation
9 tasks done
8 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.
Что изменено
Оба входа в оценивание открывают общую форму одного проекта. Из меню эксперт видит задания по всем программам; из программы — только ее проекты. Возврат сохраняет исходный список, поиск, фильтры и позицию. Пункт меню называется «оценивание».
Зачем
Согласованные макеты заменяют узкую форму внутри списка на читаемый рабочий экран. Форма в Figma, список программы.
Реализация
Отдельные маршруты списка и формы используют прежние rating repository/use cases и атомарное сохранение. Сохранены старые project_id deep links, поиск name__contains, дополнительные поля и семантика старого фильтра. Список получает собственные статусы и счетчики с API; пагинация остается по 10 с подгрузкой при прокрутке.
API
Зависит от PROCOLLAB-github/api#823: evaluation_status/evaluation_counts и комментарий до 1000 символов. Payload сохранения и обработка criteria_revision не меняются. Project/ProjectScore остается отдельным от ProgramSolution.
UI
Mont, существующие кнопки/поля/модалка и download.svg. Материалы слева, критерии справа; на телефоне одна колонка. Полные названия/описания/диапазоны; обязательный явный выбор Да/Нет; комментарий после критериев, счетчик /1000. Проверка и фокус до подтверждения; центрированное краткое подтверждение, защита повторной отправки и выхода с несохраненными ответами. Ошибки сети сохраняют ввод, конфликт критериев позволяет обновить форму и сравнить ответы. Сохраненная оценка доступна для изменения до дедлайна, в том числе после перезагрузки. Видимые кнопки 32px, мобильные области нажатия 44px.
Permissions
Не менялись: доступ, назначения, команда проекта, лимит, дедлайн и ревизия проверяются существующим сервером. Отказ API не открывает форму. Возврат допускает только кабинет или список текущей программы.
Проверки
Acceptance criteria
Не проверено
На dev проверены опубликованные файлы и совпадение трех основных исходников с Git через source maps, вход из меню и программы, статусы, фильтры и старые ссылки. Сохранение реальных оценок не повторялось: аккаунт без назначений; эти сценарии проверены на изолированном API. Production не проверялся. Дополнительный tsc всех Vitest-исходников имеет существующие ошибки вне этого изменения; штатные build/Vitest проходят.
Риски / migration
Сначала API #823 и его миграция, затем этот frontend. Production не затрагивается.