feat(tradein/v2): срок экспозиции подстрокой в карточке «Ожидаемая цена сделки» #2639
No reviewers
Labels
No labels
Fable 5 ревью
GG-форсайт
admin
analytics
auth
automation
bug
business
chore
ci
compliance
data
data-moat
docs
duplicate
dx
enhancement
feedback/max
generative
needs-discussion
needs-human
observability
pause-bots
performance
priority/p0
priority/p1
priority/p2
priority/p3
scope/backend
scope/db
scope/devops
scope/frontend
scope/qa
scrapers
security
site-finder
stage/1
stage/2
status/blocked
status/done
status/needs-analysis
status/needs-fix
status/qa
status/ready
status/review
status/wip
tech-debt
tradein
ux
week ревью 1
wontfix
ИРД
вторичка
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference: lekss361/gendesign#2639
Loading…
Add table
Reference in a new issue
No description provided.
Delete branch "feat/tradein-v2-exposure-in-result-card"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Что
В центральном блоке 02-РЕЗУЛЬТАТ карточка «ОЖИДАЕМАЯ ЦЕНА СДЕЛКИ» получила подстроку «срок продажи ~57 дн».
Почему так
Разрыв был ровно в одном аргументе:
mapSummaryполучалanalyticsData, аmapResultPanel— нет, поэтому срок экспозиции был доступен только в правом сводном рейле и отсутствовал в центральном блоке.Источник —
analytics.kpi.median_exposure_days(за нимhouse_placement_history.exposure_days, заполнено на 95%, медиана 61 д; прод отдаёт 57).Намеренно НЕ используется
est_days_on_market: он считается поdeals.days_on_market, который заполнен в 0 из 96 974 строк, и не персистится.Решения
ResultPanel.tsx,1.3fr 1.3fr 0.85fr), четвёртая колонка потребовала бы пережатия трёх существующих.exposureNote, а не переиспользованиеnote—noteуже занят сверкой с медианой ДКП и может рендериться одновременно.hasExpected: когда прогноза цены нет, карточка рендерит толькоemptyNote, и подстроке там места нет.analyticsDataдобавлена в depsuseMemo— иначе подстрока не перерисовывалась бы при догрузке аналитики (она приходит отдельным запросом, позже оценки).Проверка
npx tsc --noEmit— чисто.npm run lint— ошибок нет (2 preexisting warning'а в несвязанных файлах:CianAuthBanner,pct).Заметка про формат
Файлы
ResultPanel.tsxиpage.tsxне проходятprettier --checkи до этой правки — фронт trade-in не покрыт prettier-хуком (.pre-commit-config.yamlограничен^frontend/), CI гоняет толькоtype-check+lint. Переформатирование не делалось, чтобы не смешивать 28 строк смысла с сотнями строк пробелов.