Mixed case UUIDs are not found in the deviceConf object

bwp91 / homebridge-meross

Homebridge plugin to integrate Meross devices into HomeKit.

MIT License

194 stars 32 forks source link

Mixed case UUIDs are not found in the deviceConf object #547

Closed mrBudgie closed 6 months ago

mrBudgie commented 7 months ago

Keys are forced lower case on line 121. This PR fixes the object lookup by forcing the UUID to be lower case at the beginning of the initialiseDevice function.

Long time reader first time writer! Hope I've done this the right way...

bwp91 commented 6 months ago

@mrBudgie did you want to make these changes in your branch for the PR? or shall I make them directly? up to you!

mrBudgie commented 6 months ago

@mrBudgie did you want to make these changes in your branch for the PR? or shall I make them directly? up to you!

@bwp91 I think I’ve successfully made the changes in my branch (and sync’s the latest changes from yours) - Let me know if there’s anything else I need to do! (Apart from read the manual for GitHub - Still learning!)

bwp91 commented 6 months ago

Hi @mrBudgie

Changes are now released in version 10.2.1!

All good - never stop learning!

Now will await your next PR on here to improve something ;)

Thanks!