lekss361/gendesign

Fork 0

frontend: заменить mailto-CTA на форму лида в HeroTransparency (parent #1971) #2377

New issue

Closed

opened 2026-07-04 06:32:47 +00:00 by lekss361 · 1 comment

lekss361 commented

2026-07-04 06:32:47 +00:00

Owner

Parent: #1971
Depends on: #2376 — блокируется API-контрактом POST /api/v1/trade-in/lead. НЕ начинать реализацию до мержа backend sub-issue (или как минимум согласованного response_model).

Контекст

Единственная точка mailto-стаба во всей кодовой базе — tradein-mvp/frontend/src/components/trade-in/HeroTransparency.tsx:162-198 (leadHref, leadEnabled, CONTACT_EMAIL). Компонент маунтится через HeroSummary.tsx на src/app/page.tsx (root SPA, result-state) и ui-preview/estimate.

Scope

Заменить leadHref/mailto на inline-форму: phone (input, required) + чекбокс согласия на обработку ПД (ФЗ-152) — первый такой чекбокс в кодовой базе, готового компонента/страницы privacy-policy нет (см. Open Question ниже) — простой inline-текст без ссылки, если юзер не даст готовую формулировку/ссылку заранее.
Submit → POST /api/v1/trade-in/lead через TanStack Query useMutation (НЕ useEffect — см. .claude/rules/frontend.md), body {phone, consent, estimate_id: estimate.estimate_id, source: "result"}.
UI states: idle → submitting → success ("Заявка принята...") → error (retry). Submit заблокирован пока чекбокс не отмечен.
Убрать CONTACT_EMAIL/NEXT_PUBLIC_TRADEIN_CONTACT_EMAIL-гейтинг (leadEnabled) — форма теперь доступна всегда (уточнить у юзера, если фиче-флаг для поэтапного rollout всё ещё нужен).
После мержа backend — cd frontend && npm run codegen (обновление src/lib/api-types.ts под новый эндпоинт).

Acceptance

playwright: открыть result-state, заполнить форму, отправить → success-state; попытка отправить без чекбокса → заблокировано/client-side ошибка.

Не входит

Аналитика воронки, страница privacy-policy, CRM UI — future scope (см. комментарий на #1971).

Open questions (не блокируют старт backend #2376)

Нужна ли ссылка на страницу privacy-policy в чекбоксе согласия, или достаточно inline-текста ФЗ-152 без ссылки (страницы сейчас нет вообще)?
Сохранять ли NEXT_PUBLIC_TRADEIN_CONTACT_EMAIL-гейтинг (leadEnabled) как feature-flag для поэтапного rollout, или убрать совсем?

Референсы

tradein-mvp/frontend/src/components/trade-in/HeroTransparency.tsx (строки 59-63, 162-198)

Parent: #1971 Depends on: #2376 — блокируется API-контрактом `POST /api/v1/trade-in/lead`. НЕ начинать реализацию до мержа backend sub-issue (или как минимум согласованного response_model). ## Контекст Единственная точка mailto-стаба во всей кодовой базе — `tradein-mvp/frontend/src/components/trade-in/HeroTransparency.tsx:162-198` (`leadHref`, `leadEnabled`, `CONTACT_EMAIL`). Компонент маунтится через `HeroSummary.tsx` на `src/app/page.tsx` (root SPA, result-state) и `ui-preview/estimate`. ## Scope 1. Заменить `leadHref`/mailto на inline-форму: `phone` (input, required) + чекбокс согласия на обработку ПД (ФЗ-152) — первый такой чекбокс в кодовой базе, готового компонента/страницы privacy-policy нет (см. Open Question ниже) — простой inline-текст без ссылки, если юзер не даст готовую формулировку/ссылку заранее. 2. Submit → `POST /api/v1/trade-in/lead` через TanStack Query `useMutation` (НЕ useEffect — см. `.claude/rules/frontend.md`), body `{phone, consent, estimate_id: estimate.estimate_id, source: "result"}`. 3. UI states: idle → submitting → success ("Заявка принята...") → error (retry). Submit заблокирован пока чекбокс не отмечен. 4. Убрать `CONTACT_EMAIL`/`NEXT_PUBLIC_TRADEIN_CONTACT_EMAIL`-гейтинг (`leadEnabled`) — форма теперь доступна всегда (уточнить у юзера, если фиче-флаг для поэтапного rollout всё ещё нужен). 5. После мержа backend — `cd frontend && npm run codegen` (обновление `src/lib/api-types.ts` под новый эндпоинт). ## Acceptance playwright: открыть result-state, заполнить форму, отправить → success-state; попытка отправить без чекбокса → заблокировано/client-side ошибка. ## Не входит Аналитика воронки, страница privacy-policy, CRM UI — future scope (см. комментарий на #1971). ## Open questions (не блокируют старт backend #2376) - Нужна ли ссылка на страницу privacy-policy в чекбоксе согласия, или достаточно inline-текста ФЗ-152 без ссылки (страницы сейчас нет вообще)? - Сохранять ли `NEXT_PUBLIC_TRADEIN_CONTACT_EMAIL`-гейтинг (`leadEnabled`) как feature-flag для поэтапного rollout, или убрать совсем? ## Референсы - `tradein-mvp/frontend/src/components/trade-in/HeroTransparency.tsx` (строки 59-63, 162-198)

lekss361 referenced this issue

2026-07-04 06:32:52 +00:00

[tradein][business] Лид-флоу — тупик: CTA = mailto-стаб, нет реального POST /lead → воронка в никуда #1971

lekss361 referenced this issue

2026-07-04 07:02:12 +00:00

feat(tradein/lead): POST /api/v1/trade-in/lead — persist contact leads (#2376) #2390

bot-backend referenced this issue

2026-07-04 07:27:43 +00:00

trade-in/v2: UI/UX аудит — round 2 (системный контраст · честность чисел: дедуп/фильтр/адрес · a11y drawer+автокомплит) #2081

lekss361 referenced this issue

2026-07-04 07:33:23 +00:00

fix(tradein/v2): audit #2081 residual — 404 short-circuit, disabled-tab tooltips, stale hex token, ДКП decode #2392

bot-backend referenced this issue from a commit

2026-07-04 07:39:13 +00:00

feat(tradein/v2): replace mailto lead CTA with real form (#2377)

lekss361 referenced this issue

2026-07-04 07:39:27 +00:00

feat(tradein/v2): replace mailto lead CTA with real form (#2377) #2393

lekss361 referenced this issue from a commit

2026-07-04 08:03:03 +00:00

feat(tradein/v2): replace mailto lead CTA with real form (#2377) (#2393)

lekss361 referenced this issue

2026-07-04 08:18:30 +00:00

[tradein][v2] Форма лида (#2377) невидима на проде — смонтирована в мёртвом legacy-дереве, не в v2 #2395

bot-backend referenced this issue from a commit

2026-07-04 08:44:31 +00:00

fix(tradein/v2): смонтировать форму лида (#2377) в живое v2-дерево (#2395)

lekss361 referenced this issue

2026-07-04 08:44:46 +00:00

fix(tradein/v2): смонтировать форму лида (#2377) в живое v2-дерево (#2395) #2398

lekss361 referenced this issue from a commit

2026-07-04 08:46:13 +00:00

fix(tradein/v2): смонтировать форму лида (#2377) в живое v2-дерево (#2395) (#2398)

lekss361 referenced this issue

2026-07-04 08:57:18 +00:00

[tradein][v2] Форма лида (#2377) невидима на проде — смонтирована в мёртвом legacy-дереве, не в v2 #2395

lekss361 commented

2026-07-04 08:57:24 +00:00

Author

Owner

Closing — исходный scope (заменить mailto-CTA на форму лида) выполнен в PR #2393, но QA после деплоя обнаружил, что HeroTransparency.tsx рендерится только в недостижимом legacy-дереве (/ 307→/v2, а /v2 использует полностью другое дерево компонентов). Это не регрессия этого PR — mailto-стаб был так же мёртв, просто вопрос "лид виден реальным юзерам" оставался открытым.

Отслежено отдельным issue #2395 ("Форма лида невидима на проде — смонтирована в мёртвом legacy-дереве"), которое портировало ту же логику (переиспользуя useCreateLeadMutation as-is) в живой v2-компонент LeadForm.tsx. #2395 смержен и QA-подтверждён на живом проде (PR #2398) — форма теперь реально видна и функциональна на /trade-in/v2.

Итог: #2377 + #2395 вместе полностью закрывают исходную задачу parent-issue #1971 (lead-flow dead end).

Closing — исходный scope (заменить mailto-CTA на форму лида) выполнен в PR #2393, но QA после деплоя обнаружил, что `HeroTransparency.tsx` рендерится только в недостижимом legacy-дереве (`/` 307→`/v2`, а `/v2` использует полностью другое дерево компонентов). Это не регрессия этого PR — mailto-стаб был так же мёртв, просто вопрос "лид виден реальным юзерам" оставался открытым. Отслежено отдельным issue #2395 ("Форма лида невидима на проде — смонтирована в мёртвом legacy-дереве"), которое портировало ту же логику (переиспользуя `useCreateLeadMutation` as-is) в живой v2-компонент `LeadForm.tsx`. #2395 смержен и QA-подтверждён на живом проде (PR #2398) — форма теперь реально видна и функциональна на `/trade-in/v2`. Итог: #2377 + #2395 вместе полностью закрывают исходную задачу parent-issue #1971 (lead-flow dead end).

lekss361 closed this issue

2026-07-04 08:57:28 +00:00