Add coordinator - Githubissues

tomaae / homeassistant-truenas

TrueNAS integration for Home Assistant

Apache License 2.0

194 stars 19 forks source link

Add coordinator #72

Closed cyr-ius closed 1 year ago

cyr-ius commented 1 year ago

Proposed change

This PR is just to share my work, it contains PR #71 and replace truenas controller class with coordinator available in HA This helps to lighten and simplify the code

Type of change

[ ] Bugfix
[X] New feature
[X] Code quality improvements to existing code or addition of tests
[ ] Documentation

Checklist

[X] The code change is tested and works locally.
[X] The code has been formatted using Black.

tomaae commented 1 year ago

I know this is just a draft, but not sure I understand this change. it is bit confusing since its mixed with different PR