fix(mera/b2c): подсказка дублировала адрес, а числа в результате спорили #3138
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#3138
Loading…
Add table
Reference in a new issue
No description provided.
Delete branch "fix/3xxx-mera-b2c-copy"
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?
Два дефекта, найденных проходом клиентского пути на проде 27.08 — оба видны любому, кто открывает «Проверить квартиру».
1. Подсказка адреса печатала одно и то же дважды
Строка выпадающего списка состоит из главной строки (
label) и пояснительной (full_address). Подсказчик для дома отдаёт их совпадающими буква в букву — снято с прода:Человек видел адрес продублированным в каждой строке списка.
Показываем вторую строку только когда она отличается. Замысел (короткое имя сверху, полный адрес под ним) сохраняется, если подсказчик когда-нибудь начнёт их различать — поэтому чиню сравнением, а не удалением строки.
2. Числа в результате спорили друг с другом
Что видел клиент:
Слово «этих» указывает на 30, а рядом стоит 6. Читается как ошибка в расчёте — ровно тот случай, когда человек перестаёт верить цифрам и уходит.
Шесть — это объявления, у которых известна дата публикации; медиана считается только по ним. Теперь доля названа явно: «дата известна у 6 из 30». Когда известна у всех — доля не упоминается вовсе, чтобы «30 из 30» не шумело.
Честное имя величины сохранено. Это возраст активного объявления, а не срок продажи: выборка цензурирована, эти квартиры ещё продаются. Тест, запрещающий писать «продаются», как был, так и остался — новая формулировка его проходит.
Проверка
Два новых теста пинят именно этот класс поломки: подпись не должна содержать число, спорящее с плиткой выше, и не должна упоминать долю, когда та равна целому.
Оба дефекта найдены проходом клиентского пути на проде 27.08. ПОДСКАЗКА АДРЕСА печатала одно и то же дважды. Строка списка состоит из главной строки (`label`) и пояснительной (`full_address`), но подсказчик для дома отдаёт их совпадающими буква в букву — проверено на проде: {"label":"Свердловская область, г. Екатеринбург, ул. Малышева, д. 51", "full_address":"Свердловская область, г. Екатеринбург, ул. Малышева, д. 51"} Человек видел адрес продублированным в каждой строке выпадающего списка. Показываем вторую строку только когда она отличается: замысел (короткое имя сверху, полный адрес под ним) сохраняется, если подсказчик когда-нибудь начнёт их различать. ЧИСЛА В РЕЗУЛЬТАТЕ спорили друг с другом. Плитка сверху: «30 похожих квартир продаётся в радиусе 1 км». Плитка под ней: «столько в среднем висит объявление из этих 6». Слово «этих» указывает на 30, а стоит рядом 6 — читается как ошибка в расчёте. Шесть — это те объявления, у которых известна дата публикации; медиана считается только по ним. Теперь доля названа явно («дата известна у 6 из 30»), а когда известна у всех — не упоминается вовсе, чтобы «30 из 30» не шумело. Честное имя величины сохранено: это возраст активного объявления, а не срок продажи (тест на это как был, так и остался). Проверено: 27 тестов mera-public зелёные, tsc --noEmit чист.