ytmdl
ytmdl copied to clipboard
Include release date in prompts
Please follow the guide below
- You will be asked some questions, please read them carefully and answer honestly
- Put an
x
into all the boxes [ ] relevant to your pull request (like that [x]) - Use Preview tab to see how your pull request will actually look like
Before submitting a pull request make sure you have:
- [x] Searched the bugtracker for similar pull requests None mention 'date'
- [x] Read ytmdl coding conventions and adjusted the code to meet them None seemed applicable
- [x] Checked the code with pylama Plenty of hits, but none in song.py
- [x] Made the pull request to the unstable branch Sure.
What is the purpose of your pull request?
- [x] Bug fix
- [ ] Improvement
- [ ] New feature
Description of your pull request and other information
In short, if a song has been released on multiple albums by the same artist, the default prompt for metadata is very unhelpful:
This adds the release date to that prompt, helping to disambiguate which one we're grabbing:
To be clear, it would probably be better to include a "default" mode where we use the youtube music metadata here, but, I mean, this change unblocks me.
@elementc Thanks for the PR but adding release date to the results might not be the best option. I understand the concern that if the same song is available from various platforms, it will have different dates but what would be more helpful and visually pleasing to the user is conditional dates showing.
What we can do is if there are more than 3 results with the exact same song name, we can show the dates for all 3 of them. I feel like this can be a better option than showing it for all of them.
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs for another 7 days. Thank you for your contributions.