Skip to content

fix(mobile): show an error instead of an endless preview spinner - #9123

Merged
t3dotgg merged 2 commits into
mainfrom
fix/mobile-asset-preview-error-state
Sep 3, 2026
Merged

t3dotgg merged 2 commits into
mainfrom
fix/mobile-asset-preview-error-state

fix(mobile): treat a re-querying asset failure as loading

9cd4887
Select commit
Loading
Failed to load commit list.
MacroscopeApp / Macroscope - Correctness Check succeeded Sep 3, 2026 in 1m 52s

No issues identified (11 code objects reviewed).

• Merge Base: 18573d6
• Head: 9cd4887

Details

✅ File Path U3 Bytes Comments Posted Reason
✅ apps/mobile/src/state/asset-url-state.ts 1897 0
➖ apps/mobile/src/state/asset-url-state.test.ts 2195 Excluded by default ignore patterns
✅ apps/mobile/src/features/files/WorkspaceFilePreviewError.tsx 2232 0
✅ apps/mobile/src/features/files/WorkspaceFileVideoPreview.tsx 885 0
✅ apps/mobile/src/state/assets.ts 2732 0
✅ apps/mobile/src/features/files/ThreadFilesRouteScreen.tsx 3308 0

Billed Total: 10.00KB of diff | $0.50 (This review was charged at our per-review byte minimum of 10.00KB. Learn more here)
Saved $0.05 vs. the upper-bound estimate of $0.55 — cached code objects from prior reviews on this PR were not re-billed.