msupply-foundation / msupply-cold-chain

Android application for viewing and monitoring temperatures of fridges
GNU General Public License v3.0
3 stars 3 forks source link

Cannot remove the first digit when changing the Log Interval for each sensor #287

Open marthakeezy opened 5 months ago

marthakeezy commented 5 months ago

Describe the bug

When trying to remove the first digit of a number, the text box does not allow you to do so. For example, if I want to change from 1 minute Log Interval to 5, the field does not allow me to remove 1 (e.g. the field does not allow it to be empty).

image

To Reproduce Steps to reproduce the behavior:

  1. Go to Settings -> Available Sensors -> click on a Sensor you want to change
  2. Scroll down and click on Log Interval
  3. Try and remove the first digit to enter a different number
  4. See error

Expected behavior

The first digit should be removed in order to enter a new number, however there can be an error presented to the user if they leave the field empty (or it can automatically set to the default number mentioned in settings, e.g. 5 minutes).

Smartphone (please complete the following information):

mark-prins commented 4 months ago

cf the issue in omSupply