Skip to content
New issue

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

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

Already on GitHub? # to your account

Overhauled the tasks page #183

Merged
merged 2 commits into from
Jan 24, 2025
Merged

Overhauled the tasks page #183

merged 2 commits into from
Jan 24, 2025

Conversation

arabcoders
Copy link
Owner

This pull request includes several significant changes to the YTPTube project, including updates to the README.md file, enhancements to the HttpAPI and Emitter classes, and improvements to the configuration management. Below is a summary of the most important changes:

Documentation Updates:

  • README.md: Added new features such as multi-download support, handling live streams, scheduling downloads, supporting per-link configurations, and queueing multiple URLs. Removed outdated information about the tasks.json file. [1] [2]

New Methods and Enhancements:

API Enhancements:

  • app/library/HttpAPI.py:
    • Added add_tasks and cors_add_tasks methods to handle adding tasks via API and supporting CORS for the tasks endpoint. [1] [2]
    • Modified the __init__ method to accept a load_tasks callable and updated the tasks method to handle the absence of a tasks.json file more gracefully. [1] [2]

Configuration Management:

Main Application Enhancements:

  • app/main.py:
    • Introduced a job_item class and updated the Main class to manage cron jobs for tasks. [1] [2]
    • Enhanced the cron_runner and load_tasks methods to improve task scheduling and execution.

These changes collectively enhance the functionality, usability, and maintainability of the YTPTube project.

@arabcoders arabcoders merged commit f0ac16c into master Jan 24, 2025
11 checks passed
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant