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

Allow downloading more content from a webpage and index it #215 #525

Merged
merged 7 commits into from
Oct 28, 2024

Conversation

kamtschatka
Copy link
Collaborator

Added a worker that allows downloading videos depending on the environment variables refactored the code a bit
added new video asset
updated documentation

…p#215

Added a worker that allows downloading videos depending on the environment variables
refactored the code a bit
added new video asset
updated documentation
@MohamedBassem MohamedBassem merged commit 4a13c36 into hoarder-app:main Oct 28, 2024
4 checks passed
@kamtschatka kamtschatka deleted the video-downloading branch October 28, 2024 19:22
kamtschatka added a commit to kamtschatka/hoarder-app that referenced this pull request Nov 2, 2024
…app#525)

* Allow downloading more content from a webpage and index it hoarder-app#215
Added a worker that allows downloading videos depending on the environment variables
refactored the code a bit
added new video asset
updated documentation

* Some tweaks

* Drop the dependency on the yt-dlp wrapper

* Update openapi specs

* Dont log an error when the url is not supported

* Better handle supported websites that dont download anything

---------

Co-authored-by: Mohamed Bassem <me@mbassem.com>
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