Skip to content

test: merge master into test-branch for deployment testing#436

Merged
Movm merged 8 commits intotest-branchfrom
master
Feb 1, 2026
Merged

test: merge master into test-branch for deployment testing#436
Movm merged 8 commits intotest-branchfrom
master

Conversation

@Movm
Copy link
Copy Markdown
Collaborator

@Movm Movm commented Feb 1, 2026

Summary

  • Export download fix: Stop deleting Redis key and output file after first stream — the browser's video preview was consuming the token before the user could click download, causing "Export not found" errors
  • Standardize spinners: Replace all custom spinner implementations across the subtitler with the shared <Spinner> component, remove ~115 lines of orphaned CSS
  • Video preview aspect ratio: Set video preview to 9:16 portrait with object-fit: contain
  • Gladia logging: Log unexpected response structure before throwing error for easier debugging
  • Sentry error tracking: Pass VITE_SENTRY_DSN as Docker build arg
  • Camera scanner: Remove OpenCV/jscanify, use plain camera capture

Test plan

  • Upload a video, add subtitles, export
  • Verify video preview loads on success screen
  • Click download — should work (no "Export not found")
  • Click download again — should still work (token persists until TTL)
  • Verify all spinners render as circles, not eggs
  • Verify video preview displays in 9:16 portrait orientation

- Only set cameraState=ready from jscanify import callback
- Always capture photo, fall back to raw frame without scanner
Movm added 7 commits February 1, 2026 11:33
Drop the 8MB OpenCV CDN dependency and jscanify edge detection.
Camera now captures raw frames directly via react-webcam — simpler,
faster to load, and no WASM initialization issues.
The subtitles payload can arrive as a pre-parsed array with start/end
keys instead of the raw string format. Normalise both shapes before
passing to ASS generation to prevent export failures.
- Fix TS2322/TS2362 in processingController by wrapping unknown types with Number()/String()
- Replace @ts-ignore with @ts-expect-error in FluxImageService
- Add Sentry error tracking integration for api, web, and sites packages
- Update scraper and package dependencies
fix: resolve TypeScript errors, add Sentry error tracking & various improvements
Add VITE_SENTRY_DSN build arg to web and sites Dockerfiles so the
GlitchTip DSN is baked into the frontend bundles at build time.
Stop deleting Redis key and output file after first stream in export-download
endpoint — the browser's video preview was consuming the token before the user
could click download. Rely on the existing 1-hour TTL for cleanup instead.

Replace all custom spinner implementations across the subtitler with the shared
<Spinner> component and remove orphaned CSS. Set video preview to 9:16 portrait
aspect ratio with object-fit: contain.
@Movm Movm changed the title feat: mobile camera document scanner with jscanify test: merge master into test-branch for deployment testing Feb 1, 2026
@Movm Movm merged commit d560167 into test-branch Feb 1, 2026
14 of 17 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.

1 participant