Skip to content

fix(dashboard): synchronize mobile pairing state - #7471

Merged
houko merged 3 commits into
mainfrom
fix/dashboard-mobile-pairing-state
Aug 16, 2026
Merged

houko merged 3 commits into
mainfrom
fix/dashboard-mobile-pairing-state

Conversation

@houko

@houko houko commented Aug 15, 2026 •

Copy link
Copy Markdown
Contributor

Substantive changes

  • derive the countdown from the server expiry on every tick and stop its timer at zero
  • drive QR dimming and expired messaging from the same countdown state
  • catch QR rendering failures and retain the canvas for later pairing URIs
  • track concurrent device removals independently so only affected rows are disabled
  • replace the hardcoded dashboard URL with a typed TanStack Router link
  • add regressions for live expiry, QR rejection and recovery, concurrent removal state, and the config link
  • merge the latest origin/main

Verification

  • pnpm exec vitest run src/pages/MobilePairingPage.test.tsx (14 tests passed)
  • pnpm test (103 files, 1094 tests passed)
  • pnpm typecheck (passed)
  • pnpm lint (passed)
  • pnpm build (passed)
  • pnpm test:i18n-parity reproduced only the existing main-branch baseline: eight extra plural keys in each of pl.json and uk.json; ko.json and zh.json passed
  • git diff --check origin/main...HEAD (passed)

Out of scope

  • pairing token lifetime and refresh API behavior remain server-owned
  • device-removal transport semantics remain unchanged
  • existing Polish and Ukrainian plural-key drift remains for a separate change

@github-actions github-actions Bot added no-rust-required This task does not require Rust knowledge size/M 50-249 lines changed labels Aug 15, 2026
@houko
houko enabled auto-merge (squash) August 16, 2026 22:55
@github-actions github-actions Bot added size/L 250-999 lines changed and removed size/M 50-249 lines changed labels Aug 16, 2026
@houko
houko merged commit 5749804 into main Aug 16, 2026
39 checks passed
@houko
houko deleted the fix/dashboard-mobile-pairing-state branch August 16, 2026 23:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

no-rust-required This task does not require Rust knowledge size/L 250-999 lines changed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant