Skip to content

Remove snapshot URL sharing - #15

Merged
PengfanZ merged 1 commit into
mainfrom
codex/remove-snapshot-sharing
Jul 29, 2026
Merged

Remove snapshot URL sharing#15
PengfanZ merged 1 commit into
mainfrom
codex/remove-snapshot-sharing

Conversation

@PengfanZ

Copy link
Copy Markdown
Owner

What changed

  • remove read-only activity snapshot links, QR codes, legacy decoders, and the direct lz-string dependency
  • make local sharing focus on starting a Live activity or exporting a balances-only PNG
  • keep Live copy/QR invites, PWA link handoff, cached recovery copies, and explicit local duplication
  • clear retired #share= fragments without reading them or changing local activity data
  • update English and Simplified Chinese copy and project documentation

Why

Complete activities embedded in URLs grow with every expense and produce increasingly long links and dense QR codes. Live activities already provide the shorter, synchronized sharing path, while the balances-only export covers static results without exposing activity access.

Validation

  • npm run typecheck
  • npm run lint
  • npm run test:coverage — 260 tests and 100% coverage
  • npm run test:e2e — 12 Playwright flows
  • npx supabase test db --local — 121 pgTAP tests
  • Supabase advisors — no issues
  • npm run build:pages

@PengfanZ
PengfanZ marked this pull request as ready for review July 29, 2026 20:35
@PengfanZ
PengfanZ merged commit 38c8b53 into main Jul 29, 2026
3 checks passed
@PengfanZ
PengfanZ deleted the codex/remove-snapshot-sharing branch July 29, 2026 20:35
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