Closed jasquat closed 2 weeks ago
spiffworkflow-frontend/src/rjsf/custom_widgets/NumericRangeField/NumericRangeField.tsx (2)
`169-169`: Ensure `invalidText` and `helperText` are dynamically updated based on validation. Please verify that the `invalidText` and `helperText` for the minimum input are dynamically updated based on the validation results to ensure accurate user feedback. --- `185-185`: Ensure `invalidText` and `helperText` are dynamically updated based on validation. Please verify that the `invalidText` and `helperText` for the maximum input are dynamically updated based on the validation results to ensure accurate user feedback.spiffworkflow-frontend/src/helpers.tsx (1)
`12-12`: Validate the updated regex for numeric input. Please ensure that the updated `matchNumberRegex` correctly handles all intended numeric formats, including negative numbers, without inadvertently allowing invalid formats.spiffworkflow-frontend/src/components/CustomForm.tsx (1)
`269-270`: Refine numeric range validation to handle potential `null` values correctly. Please verify that the updated validation logic using optional chaining (`?.`) correctly handles cases where `min` or `max` values might be `null`, ensuring robust validation under all conditions.
Addresses https://github.com/sartography/spiff-arena/issues/647#issuecomment-2081717820
Summary by CodeRabbit
New Features
Bug Fixes
null
values for minimum and maximum constraints.