Fix PLT-1 temperature thresholds pinned to °C#18
Conversation
The four soil/air temperature inputs were pinned to Celsius (°C label, -10..50 range). Home Assistant converts temperature sensor states to the user's unit system, so on a Fahrenheit install the °F state is compared against a °C threshold: 'below 10' is -12 °C, so cold alerts never fire, and 'above 30' matches normal room readings, so heat alerts fire on every check. Drop the unit pin, widen the range to cover both scales, and give dual-unit starting points in each description. 🤖 Generated with [Claude Code](https://claude.com/claude-code)
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
WalkthroughThis PR updates the Soil Temperature and Air Temperature alert threshold inputs in the blueprint YAML file. Field labels are made unit-agnostic, descriptions are expanded with °C/°F guidance, the numeric slider range is widened from -10–50 to -20–130, and fixed unit_of_measurement values are removed. ChangesTemperature threshold input updates
Estimated code review effort: 1 (Trivial) | ~5 minutes Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
The four soil/air temperature inputs are labeled °C with a -10..50 range. HA converts temperature sensor states to the user's unit system, so on a °F install the state is Fahrenheit but the threshold is Celsius: "below 10" is -12 °C (cold alerts never fire) and "above 30" matches normal room readings (heat alerts fire on every repeat check once enabled).
This drops the °C pin the same way AIR-1's min/max temperature inputs handle it:
unit_of_measurementDefaults are unchanged. Existing automations keep their stored values.
🤖 Generated with Claude Code
Summary by CodeRabbit