Repository navigation
Conversation
Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01GZNK6tcaX74uMbFCJFpNPL
Настройка панели «Минимальный досмотр через SDK» (по умолчанию 0 — без порога). При выдаче показа места с наградой значение застывает в view_sec строки сессии, и досмотр раньше срока не засчитывается — как у креативов сети с API. Боевое значение задаётся только в панели. Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01GZNK6tcaX74uMbFCJFpNPL
Kennix88
added
the
release: patch
Claude
This branch has not been deployed
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
2 participants
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.
Задача: tasks/T-0029-sdk-min-view-sec.md
Что сделано
setting-catalog.ts: ключads.sdk-min-view-sec(«Минимальный досмотр через SDK»), группа «Реклама», 0–60 с, умолчание 0. Боевого значения в репозитории нет.ads.repository.ts:NewAdSession.minViewSec?;createSessionпишетview_sec = creative.viewSec ?? minViewSec ?? NULL. Условие вreport(completed)уже учитывалоview_sec— не менялось.ads.service.ts:newSessionберёт порог из настройки в момент выдачи (null, если у места нет награды или порог 0). У креатива сети с API срок свой и важнее. Слишком ранний досмотр — прежнийAdSessionClosedError(ad_session_closed, 409).ads.test.ts(порог из настройки, межстраничная без порога, креатив API без изменений, ранний/поздний досмотр, смена настройки не трогает выданные сессии);ads.integration.test.ts(живой Postgres —view_secв строке, ранний досмотр не засчитывается).docs/30-configuration-map.md— строка «Реклама».Критерии приёмки
pnpm docs:checkзелёные (см. ниже).Проверка
pnpm test— 2975 прошли, 285 пропущены. Интеграционный кейсads.integration.test.tsлокально пропущен (нет Postgres) — он написан, но прогнать его сможет только CI, прошу смотреть его результат там.pnpm lint,pnpm docs:check,pnpm task check,tsc -bиtsc -p tsconfig.tests.json— без ошибок.pnpm buildиpnpm budget: в окружении Node 22.22.0 приengines≥ 22.22.3 (engine-strict)pnpm -rне стартует; обход не делал, подтвердит CI.Метка:
release: patch.Сводка для команды
📋 СВОДКА: В DEV ВЛИТ PR #252 — ПОРОГ ДОСМОТРА РЕКЛАМЫ
Досмотр рекламы за награду через SDK сети теперь можно засчитывать не раньше порога, как уже было у наших собственных креативов.
📺 Что изменилось
• новая настройка панели «Минимальный досмотр через SDK», группа «Реклама»; 0 — без порога
• боевое значение задаётся только в панели и в репозиторий не попадает
❓ Нужно от команды
• @участник1 — задать в панели порог «Минимальный досмотр через SDK»
🤖 Generated with Claude Code
https://claude.ai/code/session_01GZNK6tcaX74uMbFCJFpNPL