brianhealey / hacs_amplipi

Adds media player support for the AmpliPi MultiZone Audio Player
MIT License
12 stars 9 forks source link

Implement controls for internet radio #6

Closed hawkeyeP closed 1 year ago

hawkeyeP commented 2 years ago

This requires adding stop to pyamplipi/pyamplipi/amplipi.py I opened another ticket for that on that repo.

(https://github.com/brianhealey/hacs_amplipi/tree/main/custom_components/amplipi)/media_player.py Line 251 Add 'internetradio'

            if self._current_stream is not None and self._current_stream.type in (
                    'internetradio',
                    'spotify',
                    'pandora'
            ):
                return SUPPORT_AMPLIPI_MEDIA
linknum23 commented 2 years ago

Now that the source that's playing reports the controls available (in 0.1.8) via info->supported_cmds this logic might now be needed.

On Sat, May 7, 2022, 20:39 hawkeyeP @.***> wrote:

This requires adding stop to pyamplipi https://github.com/brianhealey/pyamplipi/pyamplipi/amplipi.py I opened another ticket for that on that repo.

( https://github.com/brianhealey/hacs_amplipi/tree/main/custom_components/amplipi)/media_player.py Line 251 Add 'internetradio'

        if self._current_stream is not None and self._current_stream.type in (
                'internetradio',
                'spotify',
                'pandora'
        ):
            return SUPPORT_AMPLIPI_MEDIA

— Reply to this email directly, view it on GitHub https://github.com/brianhealey/hacs_amplipi/issues/6, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAEZPO6O5M3SLXTDNB6PD4LVI4ELDANCNFSM5VLEPHEQ . You are receiving this because you are subscribed to this thread.Message ID: @.***>

brianhealey commented 2 years ago

@linknum23 @hawkeyeP thanks. I updated both the pyamplipi and hacs_amplipi to support the latest firmware, and dynamically determine visibility based upon the values from the amplipi api.