athombv / homey-apps-sdk-issues

This issue tracker is for Homey Developers using the Apps SDK.
21 stars 4 forks source link

All boolean sensors are flashing red when `true`, irrespective of name #376

Open abn opened 5 days ago

abn commented 5 days ago

According to your documentation, boolean values should flash red only when they are set to true and are named with a prefix alarm_.

Booleans that are true and begin with alarm_ will flash red.

This, however, appears to be the case for all boolean values that are true. For example, I have a custom capability defined as below. This, when the value is true, is flashing red on the sensor view. This is also true, if I used a sub-capability, like onoff.subcapability in the device.

{
  "type": "boolean",
  "uiComponent": "sensor",
  "title": {
    "en": "Someone is home"
  },
  "getable": true,
  "setable": false,
  "icon": "/assets/icons/capabilities/tado_is_anyone_home.svg"
}

Is the documentation out of date (or unclear), or is this a bug? If the former, it would be great to have an option to disable this explicitly in the capability options as we do not want to alarm the user with a flashing red sensor unless it is required.

jeroenwienk commented 5 days ago

According to your documentation, boolean values should flash red only when they are set to true and are named with a prefix alarm_.

Booleans that are true and begin with alarm_ will flash red.

This, however, appears to be the case for all boolean values that are true. For example, I have a custom capability defined as below. This, when the value is true, is flashing red on the sensor view. This is also true, if I used a sub-capability, like onoff.subcapability in the device.

{
  "type": "boolean",
  "uiComponent": "sensor",
  "title": {
    "en": "Someone is home"
  },
  "getable": true,
  "setable": false,
  "icon": "/assets/icons/capabilities/tado_is_anyone_home.svg"
}

Is the documentation out of date (or unclear), or is this a bug? If the former, it would be great to have an option to disable this explicitly in the capability options as we do not want to alarm the user with a flashing red sensor unless it is required.

Do you experience this on both the web app and mobile app?

abn commented 5 days ago

I will check the the web app in a little bit, but I noticed this only in the mobile (android) app.

Edit: I can confirm that this only happens on the mobile app.

jeroenwienk commented 5 days ago

I will check the the web app in a little bit, but I noticed this only in the mobile (android) app.

Edit: I can confirm that this only happens on the mobile app.

We will fix this in the mobile app thank you for reporting.