core icon indicating copy to clipboard operation
core copied to clipboard

LaMetric: No sound when trying to do a custom sound

Open joeblack2k opened this issue 1 year ago • 1 comments

The problem

The problem The API docs from LaMetric indicate that their API supports custom MP3 URLs for notification sounds. When attempting this via notify.<device_name>, an unknown error occurs.

Diagnostics information config_entry-lametric-72e5f4e7d6428d441abd74912cbfaf07.json.txt

Additional information No response

What version of Home Assistant Core has the issue?

latest

What was the last working version of Home Assistant Core?

none

What type of installation are you running?

Home Assistant OS

Integration causing the issue

LaMetric

Link to integration documentation on our website

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

Diagnostics information

No response

Example YAML snippet

action:
  service: notify.my_lametric
  data:
    message: "Er staat iemand voor de deur!"
    data:
      icon_type: "none"
      sound:
        category: "custom"
        id: "https://home.deds.nl/~jacobshomepage/ding-dong-bell-doorbell.mp3"
        repeat: 1
      cycles: 1
      priority: "warning"
      icon: "i555"  # Voeg het gewenste icoon toe

Anything in the logs that might be useful for us?

No response

Additional information

No response

joeblack2k avatar Oct 11 '24 19:10 joeblack2k

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

Code owner commands

Code owners of lametric can trigger bot actions by commenting:

  • @home-assistant close Closes the issue.
  • @home-assistant rename Awesome new title Renames the issue.
  • @home-assistant reopen Reopen the issue.
  • @home-assistant unassign lametric Removes the current integration label and assignees on the issue, add the integration domain after the command.
  • @home-assistant add-label needs-more-information Add a label (needs-more-information, problem in dependency, problem in custom component) to the issue.
  • @home-assistant remove-label needs-more-information Remove a label (needs-more-information, problem in dependency, problem in custom component) on the issue.

(message by CodeOwnersMention)


lametric documentation lametric source (message by IssueLinks)

home-assistant[bot] avatar Oct 11 '24 19:10 home-assistant[bot]

any update on this ?

joeblack2k avatar Oct 31 '24 09:10 joeblack2k

What I get is a non-allowed ID when entering an URL. So this is not a bug as such, but a feature that has to be implemented for the "new" Lametric API.

pawal avatar Nov 24 '24 00:11 pawal

What I get is a non-allowed ID when entering an URL. So this is not a bug as such, but a feature that has to be implemented for the "new" Lametric API.

Oh wow! Is that something big? Having my LaMatric function as a doorbell is a gamechanger for me personally and replaces the need of dedicated speaker for that

joeblack2k avatar Nov 27 '24 12:11 joeblack2k

What I get is a non-allowed ID when entering an URL. So this is not a bug as such, but a feature that has to be implemented for the "new" Lametric API.

Oh wow! Is that something big? Having my LaMatric function as a doorbell is a gamechanger for me personally and replaces the need of dedicated speaker for that

Not sure, I haven't looked at the code at all. But already now you can play different types of pre-defined alarm sounds on the Lametric. Just not pointing to a URL.

pawal avatar Nov 27 '24 14:11 pawal

What I get is a non-allowed ID when entering an URL. So this is not a bug as such, but a feature that has to be implemented for the "new" Lametric API.

Oh wow! Is that something big? Having my LaMatric function as a doorbell is a gamechanger for me personally and replaces the need of dedicated speaker for that

Not sure, I haven't looked at the code at all. But already now you can play different types of pre-defined alarm sounds on the Lametric. Just not pointing to a URL.

yeah i know that, but my wife is never going to accept a different doorbell tone haha

joeblack2k avatar Nov 28 '24 20:11 joeblack2k

Hi there @joeblack2k, thanks for raising an issue. Unfortunately, this is not a bug but a feature request.

The supported values by the integration are documented here: https://www.home-assistant.io/integrations/lametric#sound

Custom sounds are supported by the newer APIs, but not implemented in the integration or library the integration uses.

../Frenck

frenck avatar Dec 12 '24 15:12 frenck