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

Fix Unknown Artist for YouTube videos #76

Merged
merged 1 commit into from
Jun 17, 2022
Merged

Fix Unknown Artist for YouTube videos #76

merged 1 commit into from
Jun 17, 2022

Conversation

supershadoe
Copy link
Contributor

According to MPRIS protocol, xesam:artist is a list of strings and not string and thus, when a youtube video is played in mpv, GNOME shell says Received faulty track artist metadata from org.mpris.MediaPlayer2.mpv; expected an array of strings, got <artistname> (string)

So just changed add_metadata_item_string to add_metadata_item_string_list and it works fine when I tested.

@hoyon hoyon merged commit 99de396 into hoyon:master Jun 17, 2022
@hoyon
Copy link
Owner

hoyon commented Jun 17, 2022

Thanks and apologies for the delay!

bmwiedemann pushed a commit to bmwiedemann/openSUSE that referenced this pull request Jun 18, 2022
https://build.opensuse.org/request/show/983511
by user mia + dimstar_suse
- Update to 0.8.1
  * Fix Unknown Artist for YouTube videos
    #gh/hoyon/mpv-mpris#76
  * Emit signal for fullscreen event
    #gh/hoyon/mpv-mpris#78

- Update to 0.7.1
  * Add test suite
  * Emit Stopped Signal on exit.
    #gh/hoyon/mpv-mpris#61
    #gh/hoyon/mpv-mpris#22
  * Set xesam:artist to channel name for YouTube videos.
    #gh/hoyon/mpv-mpris#50
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