search

zachcheatham / ha-omada

Home Assistant TP-Link Omada Integration
191 stars 26 forks source link

Update info.md #94

Open tensiondriven opened 8 months ago

tensiondriven commented 8 months ago

Loving the integration! This is a nice piece of work and far superior to the "builtin" Omada integration.

I just noticed that the content of info.md seems to be out of date with README.md. This issue is just a note to potentially update it.

I'd also love to see some more elaboration in info.md about the differences between the builtin Omada integration and this one, so n00bs would have a better chance of differentiating them. (I'd submit a PR for this but amn't able at the moment, just wanted to log this issue in case I can't get back to it.)

Another idea might be to link to info.md from README.md and then keep info.md up to date as the source of truth to prevent it from falling out of sync in the future.

While I'm ranting, it would be great to have a table of gateways/versions/firmwares that are known good / known not-so-good, since it's not too surprising that different routers/gateways/controllers etc expose radically different sensors.

Brilliant integration, much needed. I especially appriciate the configuration options to enable/disable the device trackers, etc.

fplanque commented 8 months ago

I edited and made a pull request for what you proposed.

https://github.com/zachcheatham/ha-omada/pull/95

I also want to +1 on the statements that this integration is brilliant, much needed and superior to the built in one :)