Stevens / Ivan Issue Report

PR #837 — ICUS-216: Add FAQPageJsonLd for accordion

Ссылка: https://bitbucket.org/stevens_edu/stevens_main_nextjs/pull-requests/837 Jira (sub-task): ICUS-216 — Add FAQPageJsonLd for accordion Jira (parent со спекой): https://cpcs.atlassian.net/browse/ICUS-214 — User Stories for FAQ Schema (PDF приложен) Репозиторий: stevens_edu/stevens_main_nextjs

Контекст

Один из PR, по которым со стороны Stevens / iX есть комментарии. Разбираем построчно, чтобы понять обоснованность претензий и подготовить позицию к встрече.

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

  • screenshots/ — скриншоты комментариев из PR
  • comments.md — расшифровка комментариев + наш разбор
  • summary.md — итоговая позиция по этому PR (заполнить в конце)

Статус

  • Собраны скриншоты комментариев (6 шт.)
  • Расшифрованы комментарии (5 значимых блоков)
  • Разобрана техническая суть каждого замечания
  • Сформирована наша позиция → см. summary.md

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

  • Merged: 2026-03-24 (Bemin Shaker)
  • Approved: 2026-03-23 (Bemin Shaker)
  • Цикл: 11 дней
  • Реальные претензии: 2 (Contentful environment + конфигурация поля)
  • Качество кода: "code looks great" (Michael Forbes)

Screenshots (9)

01_bemin-shaker_contentful-master-vs-develop.png
01_bemin-shaker_contentful-master-vs-develop.png
02_vercel-deploy-bot_michael-forbes.png
02_vercel-deploy-bot_michael-forbes.png
03_michael-forbes_merge-master-next-seo.png
03_michael-forbes_merge-master-next-seo.png
04a_michael-forbes_test-results.png
04a_michael-forbes_test-results.png
04b_michael-forbes_json-1.png
04b_michael-forbes_json-1.png
04c_michael-forbes_json-2.png
04c_michael-forbes_json-2.png
04d_michael-forbes_retest-after-config-change.png
04d_michael-forbes_retest-after-config-change.png
05_michael-forbes_field-config-spec-deviation.png
05_michael-forbes_field-config-spec-deviation.png
06_activity-approve-merge.png
06_activity-approve-merge.png