Код-ревью Middle Работа с разработкой обновлено
Ревью пакета передачи
Вопрос
Дизайнер передаёт в разработку экран «Список заказов». Оцените, что не так с пакетом передачи.
1. Один десктоп-макет 1440 px, мобильный «сделаем позже»
2. Таблица с пятью заказами, у всех короткие названия и суммы до 1000 ₽
3. Состояний загрузки, пустого списка и ошибки нет
4. Цвета заданы вручную hex, часть кнопок отвязана от компонентов
5. Сортировка «по умолчанию» не указана
6. В описании задачи ссылка на файл без указания нужной страницы и фрейма
7. Встреча передачи не планируется: «всё понятно из макета»
Коротко
Пакет передачи должен отвечать на вопросы разработчика до того, как он их задаст: какие состояния, какие данные, какие ширины, какое поведение и где окончательная версия.
Что должен найти ревьюер
- Ошибкастрока 1Нет адаптива: разработчик будет придумывать мобильную версию сам.
- Ошибкастрока 2Только «удобные» данные. Нужны длинные названия, большие суммы, много строк и пагинация.
- Ошибкастрока 3Нет загрузки, пустого состояния и ошибки, их придумают при реализации.
- Стильстрока 4Голые hex и отвязанные компоненты: в коде появятся новые цвета и кнопки вне системы.
- Ошибкастрока 5Не указано поведение: сортировка по умолчанию, фильтры, что по клику на строку.
- Стильстрока 6Ссылка на весь файл: разработчик может взять устаревший вариант. Нужна ссылка на конкретный фрейм со статусом готовности.
- Стильстрока 7Без короткой встречи вопросы всплывут в середине спринта.
Разбор
Пакет передачи должен отвечать на вопросы разработчика до того, как он их задаст: какие состояния, какие данные, какие ширины, какое поведение и где окончательная версия.
Ответьте сами, а не прочитайте
В тренажёре вопрос попадёт в повторения и вернётся, когда начнёт забываться. Плюс билеты с зачётом: 20 вопросов, не больше 2 ошибок.