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

Update to reflect changes in EliasKotlyar / Xiaomi-Dafang-Hacks #53

Open
sedony23 opened this issue Mar 17, 2020 · 5 comments
Open

Update to reflect changes in EliasKotlyar / Xiaomi-Dafang-Hacks #53

sedony23 opened this issue Mar 17, 2020 · 5 comments

Comments

@sedony23
Copy link

Elias updated his Xiaomi-Dafang-Hacks using different MQTT topics.

See: "move mqtt snapshot to a different topic. Fix #1300" (EliasKotlyar/Xiaomi-Dafang-Hacks@daf6de2)

Motion Detection is not working anymore... Please update your homebridge plugin :)

@mkormendy
Copy link
Contributor

@sedony23 did you make the changes on your setup, and can you create a pull request for this?

@machineglow
Copy link

machineglow commented Apr 22, 2020

I ran into this issue when I had no recordings from today (I have homekit automations that turn on recording based on these motion sensors). I had updated my Dafang FW last night so I figured something broke. Plus my homebridge log was missing all the lovely dafang mqtt messages. This lead me to this post.

I'm just a git beginner and haven't gotten git setup on my system but here's the fix in motionSensor.js:

 configureMqtt() {
    this.subscribeToMQTT('/motion')
  }

to

 configureMqtt() {
    this.subscribeToMQTT('/motion/snapshot')
  }

I just implemented the fix and restarted homebridge and recordings are live again.

@machineglow
Copy link

I tried making a branch and creating a PR but I don't have permission to do either things on this repo. oh well. Hopefully someone can make use of the info above.

@NoTrace-Kai
Copy link

motionSensor.js:

Hi there ...

where can i find this File? I searched so much, but i did not find it.

CU Kai

@jlg89
Copy link

jlg89 commented Sep 22, 2020

node_modules/homebridge-dafang/src/accessories/motionSensor.js

Line 32

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

5 participants