ruuvi / com.ruuvi.station.webui

Ruuvi webUI
BSD 3-Clause "New" or "Revised" License
4 stars 3 forks source link

Support temperature range for external temp sensor #411

Closed markoaamunkajo closed 4 months ago

markoaamunkajo commented 6 months ago

Description:

Improve alert range for temperature when using ext temp sensor

Allow adding any value to beyond current min-max limitation When edited, this value is used, with condition that min is smaller than max because app cannot know that external temperature sensor is used We will not adjust slider

markoaamunkajo commented 5 months ago

@io53 Looks like on mobile side devs (latest dev iOS build) have found a way to dynamically adjust the slider for external sensor. Would you take a look if it's also possible on web? If not, we will go with this solution.

io53 commented 5 months ago

@markoaamunkajo Sure, no problem. So if you set max value to 120, the slider should go to 120 until you have saved a smaller value?

markoaamunkajo commented 5 months ago

@io53 the min and max supported temperature range should be limited to -55 C - 150 C Please don't allow saving when value is outside this range.

Image

markoaamunkajo commented 4 months ago

Works ok for me

markoaamunkajo commented 4 months ago

QA passed