nicoduj / homebridge-deebotEcovacs

Homebridge Deebot (from Ecovacs) plugin
The Unlicense
42 stars 3 forks source link

Nick is not always there #7

Closed nicoduj closed 4 years ago

nicoduj commented 4 years ago

If it is not, handle it ;)

Found the issue and got it fixed. https://github.com/nicoduj/homebridge-deebotEcovacs/blob/master/index.js#L112 Assumes that nick is always there and tries to use it for generating a UUID which is what causes the error. Renaming in the Ecovacs app at least once causes nick to come down and fixes the issue. Might need either a default here for cases where its not yet set, or printing an error and gracefully exiting the setup instead of having Homebridge crash loop.