Updated with new path. #4957
Updated with new path. #4957
Conversation
There was a problem hiding this comment.
The examples present in in the documentation cover about the exact same topic as the linked package. Besides: that package link to custom-ui resources, so i'ts not really 'drop in'. And it's not exactly a great example (lots of commented config)
I think we should remove this as it could lead to more user confusion (and broken links in our documentation)
If needed the inline examples could be extended with another notification example, but currently they communicate the possibilities pretty well.
P.S. I just visited your GitHub page, all I can say is: WOW! 🥇
|
For now, it is a fix, since the current is invalid. @NovapaX Feel free to propose a change in a separate ticket or PR. |
|
@NovapaX - Thanks for the comments. Glad you liked the Repo.. I try hard to keep it useful. I made some changes to my package based on your suggestions. I removed all the extra commented lines from the bottom (I forgot they were there) and rearranged the Hopefully we can keep the links since i do believe my repo and the podcast are great resources for new users and little things that we can do to expose them is beneficial to all. |
Description:
Pull request in home-assistant (if applicable): home-assistant/home-assistant#
Checklist:
current. New documentation for platforms/components and features should go tonext.