SmartThingsCommunity / SmartThingsPublic

SmartThings open-source DeviceType Handlers and SmartApps code
https://developer-preview.smartthings.com/docs/devices/hub-connected/get-started
2.57k stars 89.76k forks source link

EcoBee Remote Sensor displaying Motion on overview in new app instead of Temperature #3614

Closed vseven closed 5 years ago

vseven commented 6 years ago

https://github.com/SmartThingsCommunity/SmartThingsPublic/blob/8991cd9afcd383c6b9c6ca72db716e65278d592e/devicetypes/smartthings/ecobee-sensor.src/ecobee-sensor.groovy#L19

Not sure if this is being caused by lack of meta data (assuming) but in the new Connect app EcoBee sensors are showing motion as the primary value in the overview list while in the Classic app they show temperature. Should be consistent with Classic and display temperature.

tpmanley commented 6 years ago

@workingmonk FYI

workingmonk commented 6 years ago

we've got a few more changes lined up for C2C. that should handle this bug

vseven commented 6 years ago

Sounds like this is being fixed, closing the issue.

vseven commented 5 years ago

So a recent update (within the last month) flipped this back and these are again defaulting to motion in STSC while still, correctly, showing temperature in Classic.

workingmonk commented 5 years ago

@vseven not sure what caused it to flip in the first place (possibly related to the UI overhaul), but we made the changes today to fix it back to show temperature again in the device list view.

vseven commented 5 years ago

Also as a side note the "graph" that you can click on only shows the temperatures in Celsius instead of Fahrenheit even though the region/sensor shows the temperature in Fahrenheit. Maybe this can be looked at or fixed at the same time:

EcoBeeRemoteSensor

workingmonk commented 5 years ago

@vseven that is a separate issue controlled by the UI plugin. I have filed a ticket with the plugin team to get that fixed.