No sensors after PR#71 about hue-sensors-hass

[rudders]

Can you supply the logs ?

[Djelibeybi]

I have Hue bulbs, dimmers, motion sensors. It just needs a not null check, now that I'm looking at it. Let me take a stab at a PR against master for you.

[robmarkcole]

@rudders are you happy to look into this?
@Djelibeybi I'l add a note that the last official release is the stable version
This is the problem with having not a single unit test.. :-)

[Djelibeybi]

@rudders well, there are no real logs. :)

I just added _LOGGER.warning("I got here") lines until I worked out that it was data.update() that failed and then assumed that the last commit was the problem because no-one else was complaining and I only installed it this afternoon.

[Djelibeybi]

@robmarkcole yeah, perhaps a note not to git clone unless one enjoys living on the edge. :)

[robmarkcole]

Welcome to open source!
@Djelibeybi if you can put back the offending code and generate an error that would be grea

[Djelibeybi]

FYI, it failed when trying to parse this sensor:

	"7": {
		"state": {
			"buttonevent": null,
			"lastupdated": "none"
		},
		"swupdate": {
			"state": "noupdates",
			"lastinstall": null
		},
		"config": {
			"on": true,
			"battery": 100,
			"reachable": true,
			"pending": []
		},
		"name": "Dining Room Dimmer",
		"type": "ZLLSwitch",
		"modelid": "RWL021",
		"manufacturername": "Philips",
		"productname": "Hue dimmer switch",
		"swversion": "5.45.1.17846",
		"uniqueid": "XX:XX:XX:XX:XX:XX:e3:61-02-fc00",
		"capabilities": {
			"certified": true
		}
	},

The error is simply your standard Cannot setup Hue sensors, check your config message.

And yes, we really haven't hit a button on that dimmer for a long time. :)

[rudders]

I can take a look but not for a long while - just hitting Friday night my TZ and have some family commitments. Seems like a odd issue considering my change was very localised.

Do you have a hue dimmer switch?

That’s the only code that changed.

What other hue stuff do you have?