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

[FEATURE] Youtube Playlist Download feature in Drifty CLI #631

Closed
Tarunkumarkanakam opened this issue Sep 9, 2024 · 2 comments · Fixed by #632
Closed

[FEATURE] Youtube Playlist Download feature in Drifty CLI #631

Tarunkumarkanakam opened this issue Sep 9, 2024 · 2 comments · Fixed by #632
Assignees
Labels
App 💻 Issues/Pull Requests which update Drifty Application Code EddieHub:good-first-issue Label for good-first-issue finder made by EddieHub Community feature ✨ New feature request or addition good first issue Good for newcomers
Milestone

Comments

@Tarunkumarkanakam
Copy link

Is your feature request related to a problem? Please describe.

When attempting to download a YouTube playlist using Drifty CLI, the tool fails to handle playlist URLs correctly. Although the link validation passes, the CLI is designed to extract a single video ID (from URLs like ?v=videoID) but fails when provided with a playlist URL (which uses the list parameter for the playlist ID). The error message displayed is: "YouTube video ID not found in the link," indicating that the CLI does not currently support playlist downloads.

This can be reproduced as follows

  1. Run Drifty CLI.
  2. Select "Single File Download."
  3. Provide a YouTube playlist URL (e.g., https://www.youtube.com/playlist?list=PLMC9KNkIncKtPzgY-5rmhvj7fax8fdxoj).
  4. The CLI returns: "YouTube video ID not found in the link."

Describe the solution you'd like.

  1. Update Drifty CLI to properly detect playlist URLs by looking for the list= parameter in Drifty_CLI.java
  2. Implement functionality to parse the playlist and download all videos in the playlist which is available in Drifty_GUI.java.
  3. Provide better error handling and messaging if a playlist URL is detected but can't be processed.
  4. Add an option for users to download entire playlists or select specific videos from the playlist for download would be a cool feature.

This can be referenced to issue #601

Screenshots

image

Additional information

No response

Do you want to work on this issue?

No

@Tarunkumarkanakam Tarunkumarkanakam added App 💻 Issues/Pull Requests which update Drifty Application Code feature ✨ New feature request or addition labels Sep 9, 2024
@github-project-automation github-project-automation bot moved this to Todo in Drifty Sep 9, 2024
Copy link
Contributor

github-actions bot commented Sep 9, 2024

Hello 👋! Thank you very much for raising an issue 🙌! The maintainers will get back to you soon for discussion over the issue! 🚀

Meanwhile you can also discuss about the project in our Discord Server 😀

@SaptarshiSarkar12 SaptarshiSarkar12 added this to the Drifty v2.1.0 milestone Sep 9, 2024
@SaptarshiSarkar12
Copy link
Owner

@Tarunkumarkanakam Thank you for creating this feature request.
If anyone wants to work on this issue, please comment below.

@SaptarshiSarkar12 SaptarshiSarkar12 added help wanted Extra attention and support, and contributors are needed good first issue Good for newcomers EddieHub:good-first-issue Label for good-first-issue finder made by EddieHub Community labels Sep 9, 2024
@SaptarshiSarkar12 SaptarshiSarkar12 self-assigned this Sep 20, 2024
@SaptarshiSarkar12 SaptarshiSarkar12 removed the help wanted Extra attention and support, and contributors are needed label Sep 20, 2024
@SaptarshiSarkar12 SaptarshiSarkar12 moved this from Todo to In Progress in Drifty Sep 20, 2024
@github-project-automation github-project-automation bot moved this from In Progress to Done in Drifty Sep 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
App 💻 Issues/Pull Requests which update Drifty Application Code EddieHub:good-first-issue Label for good-first-issue finder made by EddieHub Community feature ✨ New feature request or addition good first issue Good for newcomers
Projects
Status: Done
Development

Successfully merging a pull request may close this issue.

2 participants