Skip to content

Commit

Permalink
[battery_manager] React warning messages in console (#8724)
Browse files Browse the repository at this point in the history
A few number values were being passed as Strings in the props to Form elements. Change them to be numbers.

Fixes #8703
  • Loading branch information
skarya22 authored Jun 7, 2023
1 parent 9c0f84d commit 863e8a5
Showing 1 changed file with 6 additions and 6 deletions.
12 changes: 6 additions & 6 deletions modules/battery_manager/jsx/batteryManagerForm.js
Original file line number Diff line number Diff line change
Expand Up @@ -67,8 +67,8 @@ class BatteryManagerForm extends Component {
name="ageMinDays"
label="Minimum age (days)"
onUserInput={setTest}
min="0"
max="99999"
min={0}
max={99999}
required={true}
value={test.ageMinDays}
errorMessage={errors.ageMinDays}
Expand All @@ -78,8 +78,8 @@ class BatteryManagerForm extends Component {
name="ageMaxDays"
label="Maximum age (days)"
onUserInput={setTest}
min="0"
max="99999"
min={0}
max={99999}
required={true}
value={test.ageMaxDays}
errorMessage={errors.ageMaxDays}
Expand Down Expand Up @@ -134,8 +134,8 @@ class BatteryManagerForm extends Component {
label="Instrument Order"
onUserInput={setTest}
required={false}
min="0"
max="127" // max value allowed by default column type of instr_order
min={0}
max={127} // max value allowed by default column type of instr_order
value={test.instrumentOrder}
/>
<ButtonElement
Expand Down

0 comments on commit 863e8a5

Please # to comment.