danobot / entity-controller

Home Assistant Entity and lighting controller for managing devices with timers, scripts, and sun-based time restrictions.
https://danobot.github.io/ec-docs/
GNU General Public License v3.0
294 stars 40 forks source link

fix: parsing config files broke with HA 2024.2.0 release #321

Closed robchandhok closed 7 months ago

robchandhok commented 8 months ago

Description

Fixes issue introduced in HA 2024.2.0 release where test against YamlObjects types no longer work, which resulted in no configurations loading. Fix was using isinstance() instead.

Also corrected typo in debugging statement.

Very low impact change, no logic changed.

Checklist

Please open an issue before embarking on any significant pull request, especially those that add a new library or change existing tests, otherwise you risk spending a lot of time working on something that might not end up being merged into the project.

License

By submitting a patch, you agree to allow the project owners to license your work under the terms of the project license. Thank you for contributing!

Related Issues

Closes

320

dominicusmento commented 8 months ago

@danobot we need this fix asap, tx

danobot commented 7 months ago

You're a legend! The solution is so simple, yet I spent ages getting the YamlObject code to work properly for the previous fix. Glad that it's no longer necessary.