lekss361/gendesign

Fork 0

fix(tradein/front): число оценочной модели площадки подписано «рынок из N объявлений — в расчёт не входит» при тонком рынке #3349

Merged

bot-backend merged 2 commits from fix/3339-thin-market-badge into main

2026-09-05 18:12:42 +00:00

bot-backend commented

2026-09-05 17:51:30 +00:00

Collaborator

Closes #3339 (находка deep-ревью #3336). После thin-market гейта число IMV стоит рядом с headline, на который не влияет, без единой пометки.

Поправка к постановке

Карточки avito_imv в components/trade-in/v2/ нет: estimate.avito_imv читается ровно в одном месте — HeroSummary.tsx:173, где число показано дважды (bench-chip и строка price-figure-imv). Правка там, иначе подпись легла бы к UI, которого нет.

Что сделано

types/trade-in.ts: thin_market?: boolean в AvitoImvSummary;
HeroSummary.tsx: thinMarketNote() — «рынок из 3 объявлений — в расчёт не входит» (склонение: «из 1 объявления»); market_count=null → «тонкий рынок — в расчёт не входит»; false/поля нет → null, разметка не меняется. Число не гасится — оно посчитано, гасить было бы вторым враньём.

Тесты

thinMarketBadge.test.tsx (4): полный прогон Test Files 32 passed / Tests 269 passed, tsc rc=0. Фальсификация: git apply -R исходников → thinMarketNote is not a function — «функции нет», не «значение неверно»; ассерты сами по значению (строка с реальным N), ревьюеру — проверить, что число из фикстуры, не константа.

Прод

На проде тонких рынков сейчас 0 (замер 02.09) — подпись появится с первым малым городом; проверка глазами отложена до факта.

Closes #3339 (находка deep-ревью #3336). После thin-market гейта число IMV стоит рядом с headline, на который не влияет, без единой пометки. ## Поправка к постановке Карточки `avito_imv` в `components/trade-in/v2/` нет: `estimate.avito_imv` читается ровно в одном месте — `HeroSummary.tsx:173`, где число показано дважды (bench-chip и строка `price-figure-imv`). Правка там, иначе подпись легла бы к UI, которого нет. ## Что сделано - `types/trade-in.ts`: `thin_market?: boolean` в `AvitoImvSummary`; - `HeroSummary.tsx`: `thinMarketNote()` — «рынок из 3 объявлений — в расчёт не входит» (склонение: «из 1 объявления»); `market_count=null` → «тонкий рынок — в расчёт не входит»; `false`/поля нет → `null`, разметка не меняется. Число не гасится — оно посчитано, гасить было бы вторым враньём. ## Тесты `thinMarketBadge.test.tsx` (4): полный прогон `Test Files 32 passed / Tests 269 passed`, tsc rc=0. Фальсификация: `git apply -R` исходников → `thinMarketNote is not a function` — «функции нет», не «значение неверно»; ассерты сами по значению (строка с реальным N), ревьюеру — проверить, что число из фикстуры, не константа. ## Прод На проде тонких рынков сейчас 0 (замер 02.09) — подпись появится с первым малым городом; проверка глазами отложена до факта.

bot-backend added 1 commit 2026-09-05 17:51:32 +00:00

fix(mera): подписать IMV на тонком рынке — число вне расчёта

CI Trade-In / changes (pull_request) Successful in 10s

Details

CI / changes (pull_request) Successful in 12s

Details

CI Trade-In / backend-tests (pull_request) Has been skipped

Details

CI Trade-In / browser-tests (pull_request) Has been skipped

Details

CI / backend-tests (pull_request) Has been skipped

Details

CI / frontend-tests (pull_request) Has been skipped

Details

CI / openapi-codegen-check (pull_request) Has been skipped

Details

CI Trade-In / frontend-checks (pull_request) Successful in 1m51s

Details

5356d564e6

Бэкенд (#3336, thin_market) при market_count ниже порога исключает IMV из
headline, но витрина ставила число рядом с итоговой ценой молча — читалось
как расчётная величина. Подпись называет реальный N из market_count.

Closes #3339

Light1YT added 1 commit 2026-09-05 18:06:40 +00:00

fix(mera): подпись тонкого рынка доезжает до карточки оценочной модели

CI Trade-In / changes (pull_request) Successful in 13s

Details

CI / changes (pull_request) Successful in 17s

Details

CI Trade-In / backend-tests (pull_request) Has been skipped

Details

CI Trade-In / browser-tests (pull_request) Has been skipped

Details

CI / backend-tests (pull_request) Has been skipped

Details

CI / frontend-tests (pull_request) Has been skipped

Details

CI / openapi-codegen-check (pull_request) Has been skipped

Details

CI Trade-In / frontend-checks (pull_request) Successful in 2m3s

Details

d953899d92

Карточка печатает тот же recommended_price, что и hero, но берёт его из
своего endpoint и подписи не имела — рядом с итогом стояла неподписанная
копия величины, исключённой из расчёта (ровно то, что #3339 чинил в hero).

Проп note + проброс thinMarketNote(estimate.avito_imv) со страницы; подпись
тем же классом .bench-chip__sub, ему же дан max-width, чтобы длинная строка
не растягивала чип.

Тест падает при снятии рендера подписи (проверено: 1 failed / 5 passed).