Skip to content

feat(nav): add 'The Bet' to desktop top nav + drawer#284

Merged
0800tim merged 1 commit into
mainfrom
feat/nav-the-bet
Jun 4, 2026
Merged

feat(nav): add 'The Bet' to desktop top nav + drawer#284
0800tim merged 1 commit into
mainfrom
feat/nav-the-bet

Conversation

@0800tim

@0800tim 0800tim commented Jun 4, 2026

Copy link
Copy Markdown
Owner

Adds The Bet to:

  • PRIMARY_DESKTOP after Pools (desktop top bar).
  • DRAWER_PRIMARY between Pools and My Profile (app slide-out drawer).

New HouseStackIcon glyph (house on a stack of coins, matching the bet's literal subject) wired into the existing 24x24 shell-icon system.

Single i18n key nav.the_bet covers both surfaces; English fallback is "The Bet".

🤖 Generated with Claude Code

After Pools in both surfaces, so the headline-grabbing NZ$1.5M
house wager is one click away on every page. New HouseStackIcon
glyph (house perched on a stack of coins) for the entry.

  apps/web/components/shell/nav-links.tsx
    PRIMARY_DESKTOP: insert The Bet after Pools.
    DRAWER_PRIMARY:  insert The Bet between Pools and My Profile,
                     matching Tim's drawer-order spec.

  apps/web/components/shell/icons.tsx
    HouseStackIcon, lightweight SVG matching the existing 24x24
    24px-grid "baseProps" style used by the other shell icons.

Drawer label uses the same i18n key as the desktop entry so a single
'nav.the_bet' string in the locale catalogues covers both. English
fallback is 'The Bet'.

Tim 2026-06-05.
Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>

Signed-off-by: Tim Thomas <0800tim@gmail.com>
@0800tim 0800tim merged commit ae4f58b into main Jun 4, 2026
11 of 12 checks passed
@github-actions

github-actions Bot commented Jun 4, 2026

Copy link
Copy Markdown

DRY-RUN — this verdict is informational; CI is not blocked.

Auto-triage: GREEN — auto-triage clear

Risk score: 0/100

Metric Value
Files changed 2
Lines added 28
Lines removed 0
Apps touched apps/web
New dependencies 0
New 3rd-party hosts 0

No flags raised by the automated scanners. A human reviewer will still take a look.

Labels applied: area:web, auto-triage:green

Posted by @vtorn/pr-triage-bot. How this works: docs/security/01-pr-triage-process.md. Disagree with the verdict? Comment /triage override <reason> and a maintainer will re-review.

0800tim added a commit that referenced this pull request Jun 5, 2026
Tim 2026-06-05 caught the dev-only next-intl error:

  > MISSING_MESSAGE: Could not resolve `nav.the_bet` in messages
  > for locale `en`.

The nav entry was wired in PR #284 against the key `nav.the_bet`
but the locale catalogues were never updated. `safeT` catches the
miss and renders the literal "The Bet" fallback so the page never
broke, but in dev mode next-intl throws and the overlay reports the
miss every render.

Translation spread:

* Romance / Germanic / Slavic / Turkic: literal "the bet" in the
  target language (La Apuesta, Le Pari, Die Wette, La Scommessa,
  A Aposta x2, De Weddenschap, Vadet, Veddemålet, Sázka, A Fogadás,
  Bahis, Oklada x2, Garov).
* CJK: ja / ko use a brand transliteration (ザ・ベット, 더 베트);
  zh-CN uses 豪赌 ("big wager") for punch since literal "the bet"
  reads flat.
* RTL: ar / fa use the language's bet/wager noun (الرهان, شرط‌بندی).
* mi: Te Whakapae (the contest / wager).

22 catalogues updated, every existing key preserved (Python JSON
round-trip with ensure_ascii=False so non-Latin scripts stay
readable in source).

Signed-off-by: Tim Thomas <0800tim@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant