home-assistant / core

:house_with_garden: Open source home automation that puts local control and privacy first.
https://www.home-assistant.io
Apache License 2.0
73.43k stars 30.67k forks source link

Min/Max values on Plant integration not populated as attributes #70583

Closed greghesp closed 2 years ago

greghesp commented 2 years ago

The problem

Providing the min_xxxx values in the plant configuration, do not load as attributes for the entity. If this is not a bug, please consider exposing these values, as it prevents any type of useful custom card without using the patched plant component

Would be great if these could be exposed as a dict under limits in the attributes

What version of Home Assistant Core has the issue?

core-2022.4.4

What was the last working version of Home Assistant Core?

No response

What type of installation are you running?

Home Assistant Container

Integration causing the issue

Plant

Link to integration documentation on our website

https://www.home-assistant.io/integrations/plant

Diagnostics information

No response

Example YAML snippet

plant:
  Tulip:
    sensors:
      moisture: sensor.plant_moisture
      battery: sensor.plant_battery
      temperature: sensor.plant_temperature
      conductivity: sensor.plant_conductivity
      brightness: sensor.plant_brightness
    min_moisture: 15
    max_moisture: 60
    min_temperature: 6
    max_temperature: 35

Anything in the logs that might be useful for us?

No errors have been reported in the logs.

Below is an extract on the `plant.tulip` entity from above

Problem
none
Sensors
moisture: sensor.plant_moisture
battery: sensor.plant_battery
temperature: sensor.plant_temperature
conductivity: sensor.plant_conductivity
brightness: sensor.plant_brightness
Unit of measurement dict
moisture: '%'
battery: '%'
temperature: °C
conductivity: µS/cm
brightness: Lux
Moisture
55
Battery
98
Temperature
19
Conductivity
550
Brightness
3,000
Max brightness
3,000

Additional information

No response

probot-home-assistant[bot] commented 2 years ago

plant documentation plant source (message by IssueLinks)

github-actions[bot] commented 2 years ago

There hasn't been any activity on this issue recently. Due to the high number of incoming GitHub notifications, we have to clean some of the old issues, as many of them have already been resolved with the latest updates. Please make sure to update to the latest Home Assistant version and check if that solves the issue. Let us know if that works for you by adding a comment 👍 This issue has now been marked as stale and will be closed if no further activity occurs. Thank you for your contributions.