feat(web): video thumbnails, lightbox fix, and review hardening#171
Conversation
Lightbox from gallery was setting lightboxMedia to an array instead of a single object. Introduced lightboxList ref for gallery navigation while keeping lightboxMedia as a single item. Video thumbnails: added ffmpeg to Docker image and extended thumbnail generation to extract frames from video files (mp4, mkv, avi, mov, webm). Gallery endpoint now assigns thumb_url for video extensions too.
…, tests - Add separate video semaphore (limit 2) to prevent CPU/memory pressure - Consolidate extension lists into media_utils.py (single source of truth) - Fix ffmpeg to seek before input and fallback to first frame for short videos - Remove absolute path from video thumbnail log warning (PII guideline) - Add tests for _is_video, _check_ffmpeg, _generate_video_sync - Update test_rejects_non_image_file to reflect video acceptance
|
Warning Review limit reached
Your plan includes 2 reviews of capacity. Refill in 9 minutes and 3 seconds. Your organization has run out of usage credits. Purchase more in the billing tab. ⌛ How to resolve this issue?After more review capacity refills, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans have higher rate limits than trial, open-source, and free plans. In all cases, review capacity refills continuously over time. Please see our FAQ for further information. ℹ️ Review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (8)
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
🐳 Dev images published!
The dev/test instance will pick up these changes automatically (Portainer GitOps). To test locally: docker pull drumsergio/telegram-archive:dev
docker pull drumsergio/telegram-archive-viewer:dev |
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #171 +/- ##
==========================================
- Coverage 92.61% 92.55% -0.06%
==========================================
Files 23 23
Lines 6557 6599 +42
==========================================
+ Hits 6073 6108 +35
- Misses 484 491 +7
🚀 New features to boost your workflow:
|
Fixes ruff F401 lint failure in CI.
Strip file paths and folder identifiers from log lines per project logging guidelines — chat IDs are PII.
|
🐳 Dev images published!
The dev/test instance will pick up these changes automatically (Portainer GitOps). To test locally: docker pull drumsergio/telegram-archive:dev
docker pull drumsergio/telegram-archive-viewer:dev |
1 similar comment
|
🐳 Dev images published!
The dev/test instance will pick up these changes automatically (Portainer GitOps). To test locally: docker pull drumsergio/telegram-archive:dev
docker pull drumsergio/telegram-archive-viewer:dev |
1 participant
Summary
openMediaItem()was settinglightboxMediato array instead of single objectmedia_utils.py(single source of truth)Test plan