PROD: обновить новостную ленту - #376
Merged
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Что сделано
В
masterселективно перенесено финальное проверенное состояние общей новостной ленты из DEV после PR #375.Это semantic selective port, а не merge
dev → masterи не механический cherry-pick merge-коммитов.8f0202ab0937965bb548a93ca32c7fc6ff0e9242a21ea1108a2458e9108f6dd2be5f36617e7e096cmasterИсточник DEV
d4708cbc9ca7afbd245a4ff29940e2ab5dae0304e2c6d00dded1e026a9ec2850612ea8822f75e9e375f206da995fa25f87fa4ca812e3bb55fd47eb2cbdb5cf4780be0daa3e441d6b152511fcbe638d4eФинальное DEV-состояние:
bdb5cf4780be0daa3e441d6b152511fcbe638d4e.Функциональность
GET /feed/:published_atи глобальныеcountsдля шести категорий при сохранении полей пагинации.published_atпреобразуется вpublishedAt, дата выводится в форматеDD.MM.YYYY.1 2 / 3 4 / 5 6.FeedNewsи не выполняет дополнительный detail request.Enter,Space,Escape, close button, backdrop и возврат focus.16 × 16 px, числа12 pxс tabular nums, единая горизонтальная ось.feedType === project | people; legacyNewsCardна остальных surfaces не менялся.Backend/frontend contract
Angular PR ожидает совместимый ответ
GET /feed/:{ "count": 6, "next": null, "previous": null, "results": [ { "type_model": "news", "content": {}, "published_at": "..." } ], "counts": { "all": 86, "project": 56, "vacancy": 3, "news": 27, "partnerprogram": 0, "education": 0 } }Новых backend-доменов для программ и образования этот PR не добавляет.
Автоматические проверки
npm run lint:ts— exit 0, ошибок нет; 6 существующих warnings находятся вне scope PR.--checkдля 46 изменённых текстовых файлов — exit 0.npm run test:ci— 391/392 файлов и 1883/1884 теста, exit 1, 685.10 s. Единственный несвязанный timeout —export-regions-workbook.spec.ts:7(лимит 10 s).npm run test:ci -- --pool=forks— 392/392 файлов, 1884/1884 теста, exit 0, 493.55 s; тот же XLSX-тест прошёл за 642 ms.npm run build:prod— exit 0; wrapper 88.531 s, Angular build 67.164 s; initial bundle 1.40 MB raw / 314.25 kB transfer.git diff --check— exit 0; только диагностические предупреждения Git о будущем LF→CRLF.Побочное изменение
sprite.css.svgпосле Windows build восстановлено и в diff не входит. Package-файлы, scripts и workflows не менялись.Браузерная приёмка
Acceptance выполнен в Chrome 153 на реальных
FeedComponent,FeedInfoService, карточках и modal текущего порта с локальными fake use cases.Проверены viewport
1440 × 900,1280 × 900,1024 × 900,390 × 844:0 pxвезде;423.328 pxпри 1440/1280 и368.75 pxпри 1024;267.5 / 308 / 350 pxпри 1440/1280 и267.5 / 328 / 350 pxпри 1024;375 px, измеренные высоты:222 / 267.5 / 306 / 326 / 276 / 350 px;120 × 67.5 px,object-fit: cover;12 / 3 / 3 / 6 / 0 / 0, проверены active filter и mobile dropdown;6 → 12, исходные элементы сохранены, новые добавлены в конец, ключи уникальны, запрос имеетoffset=6,limit=6и выбранный type;880 × 810 px, изображение408 × 612 px; square612 × 612 px; 4:3816 × 612 px; 16:9 —840 × 472.5 px; panorama —840 × 168 px; mobile portrait —319 × 478.5 px;object-fit: contain;Enter,Space,Escape, close button, backdrop и возврат focus;234 ↔ 233;Полный протокол и точные измерения:
docs/prod-feed-news/README.mdиdocs/prod-feed-news/geometry.json.Скриншоты
Границы browser fixture
FeedComponentне передаётisOwner, поэтому owner dots недоступны в fixture; исключение owner actions из открытия modal покрыто component regression-тестами.hrefиdownload; системное скачивание не запускалось.Вне scope