search

home-assistant / home-assistant.io

:blue_book: Home Assistant User documentation
https://www.home-assistant.io
Other
4.97k stars 7.29k forks source link

HTML5 Push Notifications and Home Assistant Cloud #18319

Closed Aasikki closed 3 years ago

Aasikki commented 3 years ago

Feedback

I think the documentation could state more clearly that if I'm using Home Assistant Cloud, I'm in all good and can proceed with setting it up. Personally I had a little itch that it may not work when trying to set it up as a has cloud user, but of course it ended up working fine.

URL

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

Version

2021.6.6

Additional information

No response

thecode commented 3 years ago

Home assistant has various options for performing the same things, I don't think an integration needs to state that if you have another solution this is not needed, just to add to that, there are other options for notifications which also doesn't require Home Assistant Cloud. Integrations in Home Assistant should state what they do, this is an integration for sending HTML5 notifications for Chrome/Firefox, it is not identical to the Home Assistant cloud solution.

Aasikki commented 3 years ago

I was thinking more about the part where ssl and domain is required to set up the hmtl5 notifications and a proxy won't work. I'm still fairly new to this kind of stuff and never bothered to learn how home assistant cloud work's so I wasn't sure if it had those things included or if it was configured in a way compatible with the html notifications. I just thought it might be a good idea to tell about that in the documentation just for other people like me.

thecode commented 3 years ago

I think it is mentioned few times: image

Aasikki commented 3 years ago

Yeah so the problem is that someone new to this like me may not understand that they already have those things you rounded out in the image, when they are subscribed to home assistant cloud. For those who don't use home assistant cloud and did the manual method for ssl etc. that page is probably clear as a day.

Then again probably not many newbies even try to set this up looking at the fact it's only used by 1.8% of active home assistant installations...

This is definitely not a big deal or anything, just wanted to help in case someone else as dumb as me tries to set this up lol.

github-actions[bot] commented 3 years ago

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. If this issue is still relevant, please let us know by leaving a comment 👍 This issue has now has been marked as stale and will be closed if no further activity occurs. Thank you for your contributions.