Skip to content

QR share-app modal + finish About tab (wire UpdateCheck) - #25

Merged
astrapi69 merged 1 commit into
mainfrom
feature/qr-share
Jun 29, 2026
Merged

astrapi69 merged 1 commit into
mainfrom
feature/qr-share

Conversation

@astrapi69

Copy link
Copy Markdown
Owner

Closes #24

Now that the About tab (#17) and UpdateCheckControl (#21) are merged, this completes the About surface.

  • qrcode dependency added (approved).
  • QrCodeModal — scannable QR of a URL with copy-link / download-PNG / native-share. Lightweight custom overlay (lessons-learned: Radix portals are brittle under happy-dom).
  • ShareAppSection in the About tab opens it for the app URL (window.location.origin by default).
  • UpdateCheck wired in: UpdateCheckControl now mounts in the About tab as an "Updates" section (owner/repo placeholders to adapt).

Green

tsc --noEmit clean. Frontend Vitest 121 files / 1199 passed (+8). Production build OK (qrcode bundles fine). myapp-neutral.

…#24)

- Add qrcode dep (approved). QrCodeModal: scannable QR of a URL with
  copy-link / download-PNG / native-share, lightweight overlay (Radix
  portals are brittle under happy-dom).
- ShareAppSection in the About tab opens it for the app URL (origin).
- Wire the merged UpdateCheckControl (#21) into the About tab as an
  Updates section (owner/repo placeholders to adapt).
8 new Vitest cases. myapp-neutral.

Closes #24
@socket-security

Copy link
Copy Markdown

Review the following changes in direct dependencies. Learn more about Socket for GitHub.

Diff Package Supply Chain
Security
Vulnerability Quality Maintenance License
Addednpm/​@​types/​qrcode@​1.5.61001007481100
Addednpm/​qrcode@​1.5.410010010082100

View full report

@astrapi69
astrapi69 merged commit 0680168 into main Jun 29, 2026
9 checks passed
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.

QR-Share + finish About tab (Share app QR modal + wire UpdateCheck)

2 participants