twrecked / hass-aarlo

Asynchronous Arlo Component for Home Assistant
GNU Lesser General Public License v3.0
395 stars 78 forks source link

Just pointing out a spelling error in the readme.md file #38

Closed rpitera closed 5 years ago

rpitera commented 5 years ago

For the simplest use replace all instances of the arlo with aalro in your home-assistant configuration files.

should be

For the simplest use replace all instances of the arlo with aarlo in your home-assistant configuration files.

================= Thanks so much for this component; I'm installing today and can't wait to try it out!

twrecked commented 5 years ago

Thanks! That's actually quite an important spelling mistake.

rpitera commented 5 years ago

So far it's working great! Component is loaded and all the sensors seem to be showing up; now just have to move from my picture glance cards to your custom card.

rpitera commented 5 years ago

Oh, I just noticed another one that's important:

resources:
   - type: module
     url: /local/aarlo-glance.js

should be

 resources:
   - url: /local/aarlo-glance.js
     type: module

EDIT: I just tested this in connection with a comment I made in the PR for tracking and it seems that it doesn't make a difference. However, since every other custom card I use (and I use a LOT, lol) they use the order I suggested so you may want to adjust it just for the sake of being consistent with everyone else. BTW, this is just an awesome piece of work. This should totally replace the existing component. I am so glad that someone pointed it out to me. Totally going to tweet about this later!

twrecked commented 5 years ago

The ordering can be changed by the UI editor. At least I think I noticed that in the past.

Glad you like the component. The goal is to get it included in Home Assistant and I was going to start looking into it last week but real life (and work) got in the way.

rpitera commented 5 years ago

Like it? Hell, I tweeted about it! LOL https://twitter.com/aka_westofeast/status/1113099580705595392

Planning on doing a post in the HA subreddit as well. It's just about everything I could ask for (hoping you'll be able to set up streaming for hass.io users at some point). The fact that the alarm state isn't changing every five minutes is a godsend.

twrecked commented 5 years ago

Thanks!! It just started out as a way to fix some issues that were bugging me so I'm glad to know other people are finding it useful.

Streaming is going to need help from the some devs closer to the ffmpeg stuff. I want it fixing too because I run my main HA in a docker and it doesn't work there.

rpitera commented 5 years ago

Steve - since I have no other way to contact you, I just wanted to say that I posted about your component on the r/homeassistant subreddit and got a really good response. Here's the link to the post: https://old.reddit.com/r/homeassistant/comments/bblyjt/use_arlo_cams_in_ha_this_is_what_youve_been/

Just in case you were wondering where the traffic was coming from. LOL

twrecked commented 5 years ago

Thanks again!

Funny thing, I woke up this morning and my Google news feed was showing me the link to the Reddit article.

rpitera commented 5 years ago

LOL. I'm a mod on the r/homeautomation sub so if you want it posted there too, let me know. I mentioned it on a post in r/wyzecams in passing; someone was asking for a comparison between Arlo and Wyzecams. I've been promoting it to anyone who'll listen. I'm just so pleased with it.

Once it is in a bit wider circulation, I will put in a PR to have it added to Frenck's Awesome list. I just need to wait a bit as he's hesitant to approve anything that's too new and doesn't have enough users to recommend it. But he knows me so I'm sure that it will get added.

Hopefully, this will motivate you to keep up the great work! Cheers.

cbfreeborn commented 5 years ago

Everything in my install of this GREAT alternative to the standard arlo component went fine; however, I'm still scratching my head about where exactly

`resources:

should be located. I'm running hassio so I assumed in a /config/ui-lovelace.yaml file?

Thanks!

rccoleman commented 5 years ago

/config/ui-lovelace.yaml if you're using Lovelace in YAML mode, or paste it at the top of the raw editor if you're using it in Storage mode (the default).

twrecked commented 5 years ago

@rpitera Once again, thanks for the support! And apologies for taking so long to acknowledge it. Spring arrived here so I've been running around like a lunatic fixing things inside and outside the house.

Just FYI, I'm planning on getting a 0.5 release out this weekend then looking to try to get into the official image.

rpitera commented 5 years ago

Just to note that I have not had a single issue with your code since I installed it. Working great here and I continue to tell people about it.

twrecked commented 5 years ago

Tidying up!

And hoping everything is still working well!

rpitera commented 5 years ago

Working great and BTW I put in a PR (which was accepted and pushed) to have Aarlo added to the HA Awesome list! Congrats!