search

zachcheatham / ha-omada

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

Question: Beta releases removes users/guests #62

Closed ping-localhost closed 1 year ago

ping-localhost commented 1 year ago

I've started using the latest beta version since yesterday and it seems that users/guests have been removed in https://github.com/zachcheatham/ha-omada/commit/dd7a8c60c8d4fa7cf7c3ca673d85562ff4611acd

I like having guests/users/overall clients listed, but this is now no longer possible. Any reason why this was removed?

zachcheatham commented 1 year ago

Thanks for pointing this out. The intention is to move guests to a sensor entity.

ping-localhost commented 1 year ago

I have migrated to the sensors, but there doesn't seem to be any for guests or users, only clients (total/2.4/5). Will they be re-added?

Example diagnostic of one of my routers.

zachcheatham commented 1 year ago

Yes, I've just pushed a new beta with user and guests sensors added. They were forgotten during refactoring.

ping-localhost commented 1 year ago

Awesome! Can confirm that it works. Thank you very much!