Retro

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 / спека):

Спека обновлена 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-appmaster
  • Объём: 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 претензий письма одновременно.

Screenshots (8)

01_michael_debug-console-log.png
02_michael_build-failed-pre-merge-tasks.png
03_michael_invalid-videoobject-no-uploaddate.png
04_vercel-deploy-and-update.png
05_michael_nevermind-publishedDate.png
06_michael_700-entries-not-synced.png
07_backfill-script-discussion.png
08_pr-description-with-pre-merge-tasks.png