search

schmittx / home-assistant-eero

Eero integration for Home Assistant
MIT License
118 stars 23 forks source link

Home/Away device_tracker does not work correctly for multiple locations (when not all networks are configured as active) #6

Closed rsnodgrass closed 2 years ago

rsnodgrass commented 3 years ago

Device tracker sets home/away state regardless of which network a device is configured with. Does not seem to honor the CONF_NETWORKS setting.

rsnodgrass commented 3 years ago

I updated my fork to include extra attributes in device_trackers (e.g. which network it is on)

rsnodgrass commented 3 years ago

I have commented out a filter check to ensure clients are on networks that are configured before adding them as a tracking instance. Only because CONF_NETWORKS is not (yet) available from within EeroDeviceTracker.

@schmittx I'll let you decide how you want to expose CONF_NETWORKS into the EeroDeviceTracker so this works appropriately. Thanks!

schmittx commented 3 years ago

Sorry for the slow response here.

I want to confirm I understand the problem (since I don't have multiple networks myself).

You've got 2 networks (A and B), but only A is configured. You've then got a client within network A configured as well. The device tracker for the client works properly with regards to network A; but if the client is connected to network B then the device tracker shows Home. Is that correct?

schmittx commented 3 years ago

@rsnodgrass Any update on this? Still struggling with this problem?

schmittx commented 2 years ago

Stale, closing for now.