-
Notifications
You must be signed in to change notification settings - Fork 182
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
fix #464 by adding full date to display next to albums #474
Conversation
This commit removes the code previously responsable for filtering out duplicate album names.
Maybe you can add Another approach would be updating the UI to render the full date instead of just year. Otherwise, it will be confusing to me if I see duplicated entries. |
Here are the two options you suggested: This one doesn't fix the issue of ambiguity you addressed. This is, in my opinion, too much detail / text if you are just looking for an album. + two other options This doesn't directly show the exact distinction between the two albums, but does at least show that there is some kind of difference. Another possible solution would be to display the number of songs in an album. This is probably the most preferable solution, as it would in most cases signal a clear difference between the different versions without adding too much info or text that might not be useful to the end user just looking for an album. The only problem I have with 4 is that SimplifiedAlbum doesn't contain tracks. |
Thanks for the investigation. Really helpful. I would probably go with 2 as it might also help with filtering in case you want to find specific album. |
This change resolves the ambiguity between different Albums of the same Name.
…0#474) * fix aome510#464 by removing code to filter duplicates This commit removes the code previously responsable for filtering out duplicate album names. * Display full date next to Albums on Artist page This change resolves the ambiguity between different Albums of the same Name.
fixes #464
This PR removes the code previously responsible for filtering out duplicate album names, which had caused #464.
I have decided to not any additional text to signal the difference between the albums with the same name as:
This PR however doesn't entirely fix the bug described in the issue, as it seems that this is an upstream issue. The Spotify API doesn't return both copies of the album "Revelations" by "Audioslave" and accordingly the Spotify Client has the exact same behavior as spotfiy_player. But now the two versions of Original Fire show up as they should.
Spotify response sample:
jq '.items[].name' response.json