punxaphil / custom-sonos-card

Home Assistant custom lovelace sonos card
202 stars 22 forks source link

V7.20 Still “Apply” button and none responding “scripts” #314

Closed MNLFDF closed 7 months ago

MNLFDF commented 8 months ago

Checklist:

Release with the issue: HA 2024.3

Last working release (if known): V7.01

Browser and Operating System: Home Assistant app for iPad (v17.4) and iPhone (v17.4)

Description of problem: Installed 7.20. First it installed 7.20.beta4 (while I selected the 7.20), then Hacs notified there was a updat for the Sonos card to 7.20. I installed that. (Hacs confirmed 7.20 is installed now).

Then tried to select the second option in my list “JOE NL”. (See picture ). Then the Sonos card get me “JOE Easy” the first one on the list and again “Apply”. I can’t select any other options, because the card doesn’t seems to react on that selection. I can still pause the stream and turn up the volume tho.

And when I go to the favorites page, I can switch to another radio station.

The code I use for the dashboard:

type: custom:sonos-card
sections:
  - player
  - volumes
  - groups
  - grouping
  - media browser
predefinedGroups:
  - name: Joe Easy
    media: Joe Easy
    entities:
      - player: media_player.tv
        volume: 3
      - player: media_player.homecinema
        volume: 3
      - player: media_player.erker
        volume: 4
      - player: media_player.tafel
        volume: 3
      - player: media_player.keuken
        volume: 10
  - name: Joe NL
    media: JOE Nederland
    entities:
      - player: media_player.tv
        volume: 3
      - player: media_player.homecinema
        volume: 3
      - player: media_player.erker
        volume: 4
      - player: media_player.tafel
        volume: 3
      - player: media_player.keuken
        volume: 10
  - name: NPO Soul & Jazz
    media: NPO Soul & Jazz
    entities:
      - player: media_player.tv
        volume: 3
      - player: media_player.homecinema
        volume: 3
      - player: media_player.erker
        volume: 4
      - player: media_player.tafel
        volume: 3
      - player: media_player.keuken
        volume: 10
  - name: Radio 10
    media: Radio 10
    entities:
      - player: media_player.tv
        volume: 3
      - player: media_player.homecinema
        volume: 3
      - player: media_player.erker
        volume: 4
      - player: media_player.tafel
        volume: 3
      - player: media_player.keuken
        volume: 10
  - name: K-Joe Easy
    media: Joe Easy
    entities:
      - player: media_player.keuken
        volume: 10
  - name: K-Joe NL
    media: JOE Nederland
    entities:
      - player: media_player.keuken
        volume: 10
  - name: K-NPO Soul & Jazz
    media: NPO Soul & Jazz
    entities:
      - player: media_player.keuken
        volume: 14
  - name: K-Radio 10
    media: Radio 10
    entities:
      - player: media_player.keuken
        volume: 10
mediaArtworkOverrides:
  - ifMissing: false
    imageUrl: /local/images/sonoscard/tv1.gif
    mediaTitleEquals: TV
dynamicVolumeSlider: true
showVolumeUpAndDownButtons: true
mediaBrowserItemsPerRow: 5

IMG_6048

punxaphil commented 8 months ago

Did you use skipApplyButtonWhenGrouping: true?

MNLFDF commented 8 months ago

I add the line and indeed, no more “Apply” button. (Why the need to have a setting to turn off an extra option to push? Why is this message “apply” added in the first place, because I can’t think of anyone who want to press multiple buttons when it also can be done by just pressing one. That’s was the whole idea of grouping speakers, selecting the initial volume and stream source in the first place).

The Sonos card is going crazy now. The app (card) takes a very long time to set everything compared with the previous versions (till 7.01). Then I got an error (see pic) and the app (card) went crazy. I hear some music from one speaker, it tries to group speakers on and on, also taking speakers out of the group again (still only hear music from one speaker). Then when I open the original Sonos app, you see all speakers moving around (group/ungroup). So I had to restart Home Assistant to stop this behavior.

Also it didn’t get me the right radio station I selected. It started with the first one again.

At first when I asked you to implement the grouping speakers add initial volume setup, this card was very responsive. But after every update, I can’t tell this anymore.

Rollback to v7.0.1 and everything works as intended (but slow compared to older versions)

IMG_6129

punxaphil commented 8 months ago

Ok, I'm removing the beta. I will investigate further

MNLFDF commented 7 months ago

Ok, I'm removing the beta. I will investigate further

@punxaphil Can you tell me the status of this bug? Is this solved somewhere already in the current versions (we are now at V7.10. I am still using V7.0.1.)?

punxaphil commented 7 months ago

Please try again with latest version

MNLFDF commented 7 months ago

I still have the “apply” confirmation, but the Sonos reacts, but giving me the wrong channel. I pressed “Joe NL” and I got the first one “Joe Easy”. The reaction is also very sloooooow after you pressed “apply”. For now I am going back to V7.0.1 again. It makes no sense to me to ask for a confirmation if you already made a choice.

punxaphil commented 7 months ago

skipApplyButtonWhenGrouping: true if you don't want the apply

MNLFDF commented 7 months ago

No, this bug isn’t solved yet. Yes, the “apply” button is gone, but Sonos is still not playing the selected radio station (always the first of the list is played), I get lots of errors on my HA screen and the original Sonos app is going crazy by group speaker, ungroup them again in different order. So something fundamentally is going wrong. And it takes lots of time to handle the grouping script from the Sonos Card when I pressed the radiostation of choice. So again, back to v7.0.1. This version is working fast and does what it need to do. Unfortunately I don’t get access to the other things you build in at the Sonos Card.

IMG_6357 IMG_6358

punxaphil commented 7 months ago

Please create a specific issue for that particular problem. The title of this one mentions apply button, which is fixed.