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

Download favorites as JSON #263

Closed
wants to merge 4 commits into from
Closed

Download favorites as JSON #263

wants to merge 4 commits into from

Conversation

fc7
Copy link
Contributor

@fc7 fc7 commented Apr 18, 2024

This PR fixes a small bug in qopy.py that prevented fetching favorite tracks and artists (the API call does not take the type parameter into account and always fetches the favorite albums instead).

It also adds a method in the Qobuz core api to download all favorites as a JSON file, and exposes it in the CLI via the favs command. This is probably not very helpful for normal usage but it can be useful for post-processing or integration with other tools, or just to keep a local archive of one's favorites for reference.

@fc7
Copy link
Contributor Author

fc7 commented Apr 18, 2024

After opening this PR I now realize there is a related one for downloading one's favorites. Mine is only about fetching the metadata as JSON.

@vitiko98 it might be worth considering both PRs in parallel, wdyt?

@fc7 fc7 closed this Apr 20, 2024
@fc7
Copy link
Contributor Author

fc7 commented Apr 20, 2024

I realize it is actually a bad idea to overload the qobuz-dl cli with metadata handling. This might be better done by a separate script instead. I will reopen a new PR limited to qopy.py (bug fixes and improvements).

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.

1 participant