giachello / beoplay

Home Assistant component to control BeoPlay including TVs, Speakers and others.
23 stars 8 forks source link

media_player.py Airplay exclusion for album cover #13

Closed mbolo01 closed 1 year ago

mbolo01 commented 2 years ago

Hi, "media_player.py" has Airplay exclusion back where I think we already discussed a while ago that this exclusion is not necessary as Airplay source is able to expose the album art.

@property def media_image_url(self): """Image url of current playing media.""" _if self.source == "AirPlay": return None return self._media_url if self._media_url else None

Kind regards, Jérôme

giachello commented 2 years ago

Thats odd, don’t remember reverting it. Will check it out in the weekend maybe

On Tue, Dec 14, 2021 at 02:12 mbolo01 @.***> wrote:

Hi, "media_player.py" has Airplay exclusion back where I think we already discussed a while ago that this exclusion is not necessary as Airplay source is able to expose the album art.

@Property https://github.com/Property def media_image_url(self): """Image url of current playing media."""

_if self.source == "AirPlay": return None return self._media_url if self._media_url else None

Kind regards, Jérôme

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/giachello/beoplay/issues/13, or unsubscribe https://github.com/notifications/unsubscribe-auth/AOOHKDLPSCB7LRXFTEJRCXTUQ4KCVANCNFSM5KAOX5AQ .

--

giachello commented 1 year ago

removed it from the code.