Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Abort video download process if no videos are found #110

Merged
merged 1 commit into from
Feb 11, 2024

Conversation

Gudauu
Copy link
Contributor

@Gudauu Gudauu commented Feb 9, 2024

This pull request introduces a check for an empty video_urls list before the video downloading process begins. If no videos are found, the process is aborted with an appropriate error message, following the existing error handling pattern.

This prevents unnecessary attempts to download when there are no videos and provides clear feedback to the user.

Changes:

  • Added a check for an empty video_urls list in /Backend/main.py before downloading.
  • If the list is empty, the process is aborted with a JSON response indicating the error.

@FujiwaraChoki FujiwaraChoki merged commit 4bf5260 into FujiwaraChoki:main Feb 11, 2024
1 check 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.

2 participants