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

Top up salt alert #72

Open
steuerlexi opened this issue Oct 15, 2024 · 9 comments
Open

Top up salt alert #72

steuerlexi opened this issue Oct 15, 2024 · 9 comments

Comments

@steuerlexi
Copy link

Since the latest update, I receive frequent alerts informing me not to refill salt in my washing machine. How can I disable these alerts? I prefer to be notified only when there's no salt left.

image
@kingy444
Copy link

kingy444 commented Oct 16, 2024

there is no way to disable specific alerts at the moment, you can only turn off the alerts entirely.
You can turn off the notifications entirely if you go into CONFIGURE for the integration
image

@kingy444
Copy link

kingy444 commented Oct 16, 2024

the main issue here is the device never stops alerting (in the api) so I we cannot just treat an alert as completed. When you dismiss it HA deletes the instance of it but the API never stops telling us the issue exists.
I am able to keep it to 1 notification per alert using a unique identifier - but by definition the issue still exists (as the api is still yelling at us)

The best i think we could do is break out the above screenshotted option to each category of alert (but i dont know what all those are either)

So far have seen ALERT, WARNING and DIAGNOSTIC - so we could have 3 tick boxes instead of just one ?

@albaintor
Copy link
Owner

Hi @kingy444 I was just digging around this and just saw your latest messages.
I think that we shouldn't make it too complicated : either disable the notifications if they are annoying, and if you want some specific alerts you can configure automations for this.
Classify alerts as you suggest may be the best thing to do

@steuerlexi
Copy link
Author

Hi @albaintor @kingy444 An Automation would be fine but there is no entity for the salt alert.

@kingy444
Copy link

For now the easiest option for you is to disable all notifications and create a template sensor based on the attribute of the alerts sensor

@Sascha-L
Copy link

I would recommend to totally remove the notifications, because none of any other integration sends notifications. It's up to the user to create an automation, if he wants a specific notification. Thanks for the hint, that there is an option in the config dialog to disable the notifications, because I was also annoyed about these new notifications.

@kingy444
Copy link

Alerts was actually a feature request 😀

There are definitely some that are useful for other devices though so I think they should stay.

I'll look at enabling the granular alert settings as mentioned above, but for instance an alert that fridge door is open is important (and has ALERT status not WARNING)

I think we just need to update the README to cover disabling them as desired. I'll probably make ALERT default on and WWARNING as off

@gregersn
Copy link

Just because it is a feature request, doesn't make it right.

I have never seen any other integration giving alerts like that in notifications, and to be hones it is a very weird place to have them.

How can I disable notifications from this integration?

@bartkummel
Copy link

I would recommend to totally remove the notifications

That's not needed, since you can disable it. Other people might find it useful!

I have some questions about alerts:

  • Would there be an option to translate/localize the alerts? And apply some filters to them so they look nicer? In templates you can do things like |replace("_"," ")|capitalize to convert it from SCREAMING_SNAKE_CASE to something more human friendly.
  • How should I interpret the following:
    Alert: DISH_ALARM_RINSE_AID_LOW
    Severity: WARNING
    Status: NOT_NEEDED
    
    The rinse aid is low, it's a warning, but then... something is not needed? I assume it's meant to say that immediate action is not needed or something? It's a bit confusing if put like this...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

6 participants