Retro

PR #832 — IX-SIT60: Add schema for pages

Ссылка: https://bitbucket.org/stevens_edu/stevens_main_nextjs/pull-requests/832 Jira: IX-SIT60 — Add schema for pages Репозиторий: stevens_edu/stevens_main_nextjs

Замечание по нумерации: PR #842 был помечен как IX-SIT60 ("Add schema for youtube"), а этот PR #832 — тоже IX-SIT60 ("Add schema for pages"). Скорее всего, IX-SIT60 — общий тикет/эпик, под который заведено несколько PR (по разным компонентам schema). Уточнить у Ивана / в Jira, как это структурировано.

Контекст

Помечен как "следующий большой" — то есть значимый по объёму PR. Тематически продолжение работы по Schema/SEO (после #837 FAQ, #842 youtube → теперь pages).

Спека и даты (задача IX-SIT60)

  • 2026-02-17 — в задаче доступна спека 📄Stevens_Schema_Templates.odt (шаблоны schema / примеры типов страниц; тот же документ может фигурировать как Stevens_Schema_Templates (1).odt при экспорте/копии).
  • 2026-02-19 — старт работ по задаче.
  • 2026-02-21 — открыт PR #832 (через 2 дня после старта работ).

Структура папки

  • — скриншоты комментариев из PR
  • spec/ — спецификация (если есть отдельная по pages schema)
  • — расшифровка комментариев + наш разбор
  • — итоговая позиция

Статус

  • Собраны скриншоты (12 шт.)
  • «Schema templates doc» / Stevens_Schema_Templates.odt (Bemin): не отдельный фактор — документ в задаче с 17.02, работа начата 19.02; квест «найти документ» снят. Разбор Comment #1 — про покрытие типов в PR, не про доступность спеки.
  • Расшифрованы комментарии
  • Сформирована позиция → см.

Финальный результат

  • Status: OPEN
  • Цикл: 2+ месяца (21.02 → 24.04+)
  • Объём: ≥27 коммитов в 9+ батчах
  • REQUESTED CHANGES от Michael Forbes (31.03)
  • 9 значимых замечаний, 8 из 9 обоснованы
  • Самый проблемный PR. Концентрирует все 4 пункта письма Alexis.
  • Alexis уже 23.02 сигнализировал про pre-deployment steps — за 2 месяца до письма от 29.04. Сигнал был, не отработан.
  • Повторяющаяся ошибка про invalid VideoObject — та же ошибка, что в PR #842.

Screenshots (12)

01_open-and-vercel-bot.png
02_bemin_minisite-right-nav-missing.png
03_alexis_hold-and-pre-deployment-question.png
04_bemin_contact-pages-postaladdress-rich-text.png
05_commits-march-batches.png
06_michael_invalid-videojsonld-hero-requested-changes.png
07_michael_video-with-caption-whitespace.png
08_michael_featured-video-whitespace-conflict.png
09_bemin_undefined-base-url.png
10_bemin_hardcoded-news-prefix.png
11_bemin_image-credit-field-events.png
12_bemin_image-credit-field-program.png