fix(tradein): white-label без logo не показывает «Мера»-глиф (#1081) #1097

Merged
bot-reviewer merged 1 commit from fix/1081-whitelabel-mark into main 2026-06-06 19:09:14 +00:00
Owner

Closes #1081 (follow-up из ребрендинга #1079).

Проблема

White-label клиент с настроенным брендом, но без валидного logo_url, рендерил наш фирменный <MeraMark /> рядом с именем клиента → лик нашего бренда на клиентской шапке (напр. «[М-глиф] ПРАКТИКА»).

Fix

  • Topbar.tsx white-label fallback: <MeraMark />{brand.name.slice(0,1).toUpperCase()} (нейтральный плейсхолдер — первая буква имени клиента в том же .brand-mark бейдже). <MeraMark /> остаётся ТОЛЬКО в дефолтной (brand === null) ветке — там это наш продукт, корректно.
  • OfferCard.tsx: добавлены inline-комментарии у видимых «Trade-In» (механика сделки, не бренд) — чтобы будущий рефактор не переименовал по ошибке (#1079).

Verify

tsc --noEmit 0 ошибок · next build green (12/12).

Closes #1081 (follow-up из ребрендинга #1079). ## Проблема White-label клиент с настроенным брендом, но без валидного `logo_url`, рендерил наш фирменный `<MeraMark />` рядом с именем клиента → лик нашего бренда на клиентской шапке (напр. «[М-глиф] ПРАКТИКА»). ## Fix - `Topbar.tsx` white-label fallback: `<MeraMark />` → `{brand.name.slice(0,1).toUpperCase()}` (нейтральный плейсхолдер — первая буква имени клиента в том же `.brand-mark` бейдже). `<MeraMark />` остаётся ТОЛЬКО в дефолтной (`brand === null`) ветке — там это наш продукт, корректно. - `OfferCard.tsx`: добавлены inline-комментарии у видимых «Trade-In» (механика сделки, не бренд) — чтобы будущий рефактор не переименовал по ошибке (#1079). ## Verify `tsc --noEmit` 0 ошибок · `next build` green (12/12).
lekss361 added 1 commit 2026-06-06 19:07:04 +00:00
fix(tradein): white-label fallback shows client initial, not MeraMark (#1081)
All checks were successful
CI / changes (push) Successful in 9s
CI / backend-tests (push) Has been skipped
CI / changes (pull_request) Successful in 6s
CI / backend-tests (pull_request) Has been skipped
19b28e3e13
In the no-logo_url fallback branch replace <MeraMark /> with the first
letter of the client brand name so our glyph never appears in a
white-label header.  Default (brand===null) branch is unchanged.
Add inline comments in OfferCard to guard Trade-In mechanic strings
against unintentional rebrand sweep (#1079).
bot-reviewer reviewed 2026-06-06 19:09:07 +00:00
bot-reviewer left a comment
Collaborator

APPROVE

Закрывает #1081 ровно по рекомендации ревью #1079:

  • White-label-no-logo fallback: <MeraMark />{brand.name.slice(0,1).toUpperCase()} (нейтральный инициал в том же .brand-mark бейдже). MeraMark остаётся ТОЛЬКО в дефолтной (brand===null) ветке — наш продукт, корректно. Бренд больше не утекает в клиентскую шапку.
  • OfferCard: добавлены {/* intentional: механика сделки, не бренд (rebrand #1079) */} у трёх «Trade-In» — будущий рефактор не переименует по ошибке.
  • tsc 0 + build green 12/12.

Edge: brand.name="" → пустой бейдж — но white-label бренд всегда с именем, cosmetic.

Merge → deploy-tradein verify (title-flip уже «Мера»; white-label fallback не наблюдаем публично без brand-сессии).

<!-- gendesign-review-bot: sha=19b28e3 verdict=approve --> ✅ **APPROVE** Закрывает #1081 ровно по рекомендации ревью #1079: - White-label-no-logo fallback: `<MeraMark />` → `{brand.name.slice(0,1).toUpperCase()}` (нейтральный инициал в том же `.brand-mark` бейдже). `MeraMark` остаётся ТОЛЬКО в дефолтной (`brand===null`) ветке — наш продукт, корректно. Бренд больше не утекает в клиентскую шапку. - OfferCard: добавлены `{/* intentional: механика сделки, не бренд (rebrand #1079) */}` у трёх «Trade-In» — будущий рефактор не переименует по ошибке. - tsc 0 + build green 12/12. Edge: `brand.name=""` → пустой бейдж — но white-label бренд всегда с именем, cosmetic. Merge → deploy-tradein verify (title-flip уже «Мера»; white-label fallback не наблюдаем публично без brand-сессии).
bot-reviewer merged commit e792bed325 into main 2026-06-06 19:09:14 +00:00
Sign in to join this conversation.
No reviewers
No milestone
No project
No assignees
2 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: lekss361/gendesign#1097
No description provided.