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

FullyKiosk media player entity stays in state 'playing' forever #83261

Closed
bcutter opened this issue Dec 4, 2022 · 27 comments · Fixed by #105762
Closed

FullyKiosk media player entity stays in state 'playing' forever #83261

bcutter opened this issue Dec 4, 2022 · 27 comments · Fixed by #105762

Comments

@bcutter
Copy link

bcutter commented Dec 4, 2022

The problem

Once the media player entity switches to playing, it keeps that state like forever. That's a problem as it is used massively for notifications (so no one is clicking "stop" regularly) and therefore renders the usage of the media player entity a bit useless.

In detail: once the media player entity switches to playing, it won't resume from that state until

  • a) HA has been restarted
  • b) the integration has been reloaded
  • c) the stop button has been pressed

What version of Home Assistant Core has the issue?

core-2022.9.7

What was the last working version of Home Assistant Core?

No response

What type of installation are you running?

Home Assistant OS

Integration causing the issue

fully_kiosk

Link to integration documentation on our website

https://www.home-assistant.io/integrations/fully_kiosk

Diagnostics information

No response

Example YAML snippet

No response

Anything in the logs that might be useful for us?

No response

Additional information

cgarwood/homeassistant-fullykiosk#118 (comment)

@home-assistant
Copy link

home-assistant bot commented Dec 4, 2022

Hey there @cgarwood, mind taking a look at this issue as it has been labeled with an integration (fully_kiosk) you are listed as a code owner for? Thanks!

Code owner commands

Code owners of fully_kiosk can trigger bot actions by commenting:

  • @home-assistant close Closes the issue.
  • @home-assistant rename Awesome new title Change the title of the issue.
  • @home-assistant reopen Reopen the issue.
  • @home-assistant unassign fully_kiosk Removes the current integration label and assignees on the issue, add the integration domain after the command.

(message by CodeOwnersMention)


fully_kiosk documentation
fully_kiosk source
(message by IssueLinks)

@issue-triage-workflows
Copy link

There hasn't been any activity on this issue recently. Due to the high number of incoming GitHub notifications, we have to clean some of the old issues, as many of them have already been resolved with the latest updates.
Please make sure to update to the latest Home Assistant version and check if that solves the issue. Let us know if that works for you by adding a comment 👍
This issue has now been marked as stale and will be closed if no further activity occurs. Thank you for your contributions.

@bcutter
Copy link
Author

bcutter commented Mar 4, 2023

No one ever took a look at this in 3 months.

So let's keep this open and wait another 3 months. Thanks bot.

@github-actions github-actions bot removed the stale label Mar 5, 2023
@TheAlphaLaw
Copy link

No one ever took a look at this in 3 months.

So let's keep this open and wait another 3 months. Thanks bot.

I am seeing the same issue here.

@mheath
Copy link
Contributor

mheath commented Apr 26, 2023

While FKB has an API to play sound media, it does not have an API to say if it has finished. Given the limitations of the FKB API, I don't see a way to resolve this issue.

@erkr
Copy link

erkr commented May 6, 2023

Same issue, will ask FKB support to extend the API 🤞

@bcutter
Copy link
Author

bcutter commented May 6, 2023

Same issue, will ask FKB support to extend the API 🤞

Thank you. Feel free to link it here.

@erkr
Copy link

erkr commented May 6, 2023

I sent an email, but will report back on their response

@erkr
Copy link

erkr commented May 7, 2023

Hi @mheath, @bcutter

In got a response from FKB support. Please help me with an answer, as this is bestond my knowledge:

Hello Eric,

without knowing anything about HA I could read this on one of the threads:

> While FKB has an API to play sound media, it does not have an API to say if it has finished.

Which Fully Kiosk API is used exactly to play sound media?

Alexey

--
Fully Kiosk Solutions for Android
https://www.fully-kiosk.com

@mheath
Copy link
Contributor

mheath commented May 7, 2023

We use the REST API /?cmd=playSound.

@erkr
Copy link

erkr commented May 8, 2023

Forwarded you answer. Thanks!

@erkr
Copy link

erkr commented May 8, 2023

@mheath got this answer. Sounds promising:

Hello Eric,

adding these information to the cmd=getDeviceInfo&type=json REST API was not a big deal. You can test or let the developers test with the latest beta version.

https://www.fully-kiosk.com/files/2023/05/Fully-Kiosk-Browser-v1.51-beta-1.apk

The new fields are audioVolumes and soundUrlPlaying

Alexey 

@mheath
Copy link
Contributor

mheath commented May 10, 2023

Thanks for reaching out to Fully Kiosk @erkr. Very cool that they're so responsive. Adding soundUrlPlaying to device info should make fixing this issue fairly simple.

Adding MQTT events for sound start/stop will allow us to make the media control more responsive for those with MQTT enabled. (There are already MQTT volume events.)

@TheAlphaLaw
Copy link

Thanks for reaching out to Fully Kiosk @erkr. Very cool that they're so responsive. Adding soundUrlPlaying to device info should make fixing this issue fairly simple.

Adding MQTT events for sound start/stop will allow us to make the media control more responsive for those with MQTT enabled. (There are already MQTT volume events.)

I have MQTT enabled but can't figure out how to send media to any device with Fully Kiosk. I'm able to use the media player through home assistant, but it can have a major delay. Any tips on how to config?

@kernelpanic85
Copy link
Contributor

I'm also having this issue. Did Fully Kiosk implement the change needed? Does the HA side just need an update at this point?

@mheath
Copy link
Contributor

mheath commented Aug 10, 2023

The HA side still needs to be updated to see when the media has stopped playing, volume, and MQTT support still hasn't been added. There aren't any MQTT features in FKB that will significantly help with the media player.

I don't think the changes we talked about previously will fix the "major delay" issue.

@erkr
Copy link

erkr commented Aug 11, 2023

@mheath thanks for the heads up. Looking forward to have the stopped playing and volume added👍

@issue-triage-workflows
Copy link

There hasn't been any activity on this issue recently. Due to the high number of incoming GitHub notifications, we have to clean some of the old issues, as many of them have already been resolved with the latest updates.
Please make sure to update to the latest Home Assistant version and check if that solves the issue. Let us know if that works for you by adding a comment 👍
This issue has now been marked as stale and will be closed if no further activity occurs. Thank you for your contributions.

@kernelpanic85
Copy link
Contributor

Still an issue for sure. Anybody have luck looking into this?

@issue-triage-workflows
Copy link

There hasn't been any activity on this issue recently. Due to the high number of incoming GitHub notifications, we have to clean some of the old issues, as many of them have already been resolved with the latest updates.
Please make sure to update to the latest Home Assistant version and check if that solves the issue. Let us know if that works for you by adding a comment 👍
This issue has now been marked as stale and will be closed if no further activity occurs. Thank you for your contributions.

@bcutter
Copy link
Author

bcutter commented Feb 7, 2024

#83261 (comment)

@erkr
Copy link

erkr commented Feb 7, 2024

Still the case

@sophof
Copy link
Contributor

sophof commented Feb 10, 2024

It sounds like @mheath knows how to fix this, is that correct? Reading this back (as I just ran into this issue) it appears that is what everyone assumes at least?
If not I could try digging in (but I'd rather not)

@cgarwood
Copy link
Member

There is a PR open to fix, just waiting for review: #105762

@erkr
Copy link

erkr commented Feb 10, 2024

Great, thanks! I really hope it gets merged in soon😊

@bcutter
Copy link
Author

bcutter commented Feb 14, 2024

Great! Hopefully #105762 will make it to a stable release soon. Really looking forward to read about this in the release notes :-)

@erkr
Copy link

erkr commented Feb 14, 2024

@cgarwood thanks for picking this up and looking forward to have it released 😊

@github-actions github-actions bot locked and limited conversation to collaborators Mar 15, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

7 participants