-
Notifications
You must be signed in to change notification settings - Fork 40
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
Is this repo still maintained? #40
Comments
I am thinking of forking the repo and adding al those good PR's into it. And then keep that actively maintained. Anyone has any ideas on this? cc @vmax77 |
i have one here which has a bunch of PRs merged, bit of clean up and support for both running as a service or via a scheduled task. I'm not maintaining but if you just wanted something to grab quickly until this is updated, go for it. |
I've created a copy of the repo (with a credit section for the author here in the README) and will be maintaining it, PRs are welcomed: |
Just installed your version, thanks a lot! Do you think you will be able to integrate the webcam PR? Would love that! Didn't get it to work on my cut paste actions. |
With your repro the settings file contains:
Which imiplies there is a Heartbeat binary sensor. I cannot find a pointer in the readme to create this. Is this just a binary sensor called 'teams monitoring'? |
Yeah I will review the camera monitoring and see about adding it. For the Heartbeat sensor, this was from the original source, and it does seem to do absolutely nothing so I will remove it. |
No worries, take your time. Just FYI, it works fine for me as a service. Main issue why it wasn't working for me was the running user. I have now changed it from local system to my own username and it runs fine. |
After reading up on the other conversations it makes sense that this would fix it, since it was the %appdata% that was yielding the inccorect path. |
Alright I think I've got the camera working fairly well, it's not ideal since it does not seem to generate a log entry when it comes on/goes off so another entry needs to trigger the service to check the registry. I also ended up with your approach, the installer now asks for the windows password to install it under the logged-in user's account. If you have any issues (or improvements to be made) you are welcome to open an issue or a PR on the new repo page! |
FYI I released a rewrite in Rust that uses their WebSockets API, it connects to HA and MQTT: https://github.com/AntoineGS/teams-status-rs |
There are some pull requests from many months ago, with some nice fixes and features. I'm wondering if the repo is still maintained?
The text was updated successfully, but these errors were encountered: