Skip to content

fix: resolve multiple bugs across backend and frontend#1

Open
vmansus wants to merge 2 commits into
mainfrom
claude/review-repository-C0Fbo
Open

fix: resolve multiple bugs across backend and frontend#1
vmansus wants to merge 2 commits into
mainfrom
claude/review-repository-C0Fbo

Conversation

@vmansus
Copy link
Copy Markdown
Owner

@vmansus vmansus commented Apr 12, 2026

Backend:

  • Add missing remote_components to subtitle_opts in youtube.py
  • Sync video_info.py and youtube_fetcher.py to use tv player_client
    and remote_components (consistent with youtube.py)
  • Fix any → Any type hints in task_executor.py
  • Remove duplicate OCR entries in api/main.py options_dict
  • Remove duplicate import os in pipeline.py

Frontend:

  • Replace raw fetch() with taskApi.delete() in Dashboard for
    consistent API usage and proper error handling
  • Add taskApi.delete() method to api.ts
  • Show error alert on delete failure instead of silent catch

https://claude.ai/code/session_01EdyoR2GQjoL2Gw4tCB8r6N

claude added 2 commits April 12, 2026 16:28
Backend:
- Add missing remote_components to subtitle_opts in youtube.py
- Sync video_info.py and youtube_fetcher.py to use tv player_client
  and remote_components (consistent with youtube.py)
- Fix any → Any type hints in task_executor.py
- Remove duplicate OCR entries in api/main.py options_dict
- Remove duplicate import os in pipeline.py

Frontend:
- Replace raw fetch() with taskApi.delete() in Dashboard for
  consistent API usage and proper error handling
- Add taskApi.delete() method to api.ts
- Show error alert on delete failure instead of silent catch

https://claude.ai/code/session_01EdyoR2GQjoL2Gw4tCB8r6N
The Settings page uses tiktok.max_publish_age but the TikTokSettings
interface was missing this field, causing TypeScript compilation to fail.

https://claude.ai/code/session_01EdyoR2GQjoL2Gw4tCB8r6N
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.

2 participants