PR #842 — IX-SIT60: Add schema for youtube
Ссылка: https://bitbucket.org/stevens_edu/stevens_main_nextjs/pull-requests/842 Jira (PR-задача): IX-SIT60 — Add schema for youtube Jira (parent / спека):
- https://cpcs.atlassian.net/browse/ICUS-211 — YouTube publish date sync (Contentful App)
- https://cpcs.atlassian.net/browse/ICUS-212 — то же самое (текст совпадает)
Спека обновлена 2026-03-12 (Option 2: Contentful App, по решению Michael Forbes от 2026-03-10). PR открыт 2026-03-31 — через 19 дней после обновления спеки.
Репозиторий: stevens_edu/stevens_main_nextjs
Owner / ответственность по задаче
- Постановка / scope-владелец: Olexandr (Sasha) Fedorenko — он вёл задачу IX-SIT60 / ICUS-211(212), решал scope (Option 2 — Contentful App, тригеры, поведение поля
youtubeUploadDate, формат данных, миграция существующих 700 entries и т.д.). - Иван: коммитил код в этот PR. По части scope-решений (что выводится в schema, когда тригерится sync, как обрабатываются существующие 700 entries) вопросы не к Ивану, а к Саше.
- Что остаётся за Иваном в этом PR: гигиена коммита (debug
console.log), реактивность на ревью, общая дисциплина PR description.
На встрече разделять: "это вопрос к Саше" (scope/архитектура Contentful App) vs "это к Ивану" (гигиена кода / console.log / реакция на ревью). См. — расставлены маркеры.
🆕 Репо Contentful App найден
stevens_main_ctfapps, PR #1 "Initial Contentful app import" — https://bitbucket.org/stevens_edu/stevens_main_ctfapps/pull-requests/1
- Статус: OPEN с 2026-04-22 (~9 дней), branch
contentful-app→master - Объём: 36 файлов, 1 коммит
- Ревьюер Michael Forbes добавлен 2 дня назад (~29-30.04)
- Description: "Initial import of the Contentful app project, including source code, config, tests, and backfill tooling" — то есть backfill для 700 entries уже включён в App (после запроса Michael).
- Автор PR —
Alex.Rudenok, не Sasha Fedorenko. Может быть тот же человек под другим логином, либо отдельный разработчик. Уточнить на встрече.
Контекст
Следующий PR для разбора — добавление schema (структурированных данных) для YouTube-видео. Тематически близок к PR #837 (там была FAQ schema, тут — VideoObject schema, скорее всего).
Структура папки
- — скриншоты комментариев из PR
spec/— спецификация (если найдётся в Jira)- — расшифровка комментариев + наш разбор
- — итоговая позиция (заполнить в конце)
Статус
- Собраны скриншоты комментариев (8 шт.)
- Найдена спека → ICUS-211 / ICUS-212, см.
- Расшифрованы комментарии
- Сформирована позиция → см.
- YouTube API key — в git не попадал (PR #842, репо Contentful App); срочная проверка снята
Финальный результат PR
- Status: OPEN на 2026-04-16 (не замержен)
- Цикл: 16+ дней (открыт 31.03)
- Реальные претензии: 4 (debug-код, отсутствие pre-merge tasks, невалидная schema, ~700 entries без миграции)
- Этот PR — почти точная иллюстрация письма Alexis от 29.04. На нём сошлись 3 из 4 претензий письма одновременно.