Seeding Input Optional Labor Configuration Firefox Input Bug

[JingyuMarcelLee]

Description of the Issue

When labor is set to "Optional" on the configuration page, the user should be able to either not enter any labor data or enter both labor data. Because the firefox HTML input form allows users to put non-numeric characters even though its type is set to number and simply returns an empty string for the value, there is no way to distinguish between when a user puts invalid input (thus firefox returning "") or user clearing the input after putting something in there (initially it is set to null, then user puts something in it, then deletes it, making it an empty string ""). Thus, whenever the user puts in an invalid input, the feature implemented at #541 stops working.

Steps to Reproduce

1. Set the configuration to "optional" on the configuration page
2. Go to the Seeding input and put all the values except labor-related fields to test the submit button.
3. Put a valid value in one of the fields
   
4. Put an invalid value on the other field
5. Clear the field with a valid value. Now the submit button is highlighted and the field with wrong input is not highlighted

Suggested Solution

Need research.

[Panktivpatel]

I would like to work on this please!

[braughtg]

@Panktivpatel Welcome. It would be great to see what you can do with this issue.

Note: If you are in a class that is using the "GitKit" and an assignment asked you to comment on an issue for your course be sure you are using the correct "upstream" for your course. This is the main upstream for FarmData2. We welcome your contributions. But I wanted to be sure this was what you intended.

[Panktivpatel]

Hello, Yes i have been assigned to comment on an issue for my course

…

On Mon, Jan 29, 2024 at 2:13 PM Grant Braught ***@***.***> wrote: @Panktivpatel <https://github.com/Panktivpatel> Welcome. It would be great to see what you can do with this issue. Note: If you are in a class that is using the "GitKit" and an assignment asked you to comment on an issue for your course be sure you are using the correct "upstream" for your course. This is the main upstream for FarmData2. We welcome your contributions. But I wanted to be sure this was what you intended. — Reply to this email directly, view it on GitHub <#546 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/A65CZZNOKKEVQRLAV4VV5Y3YQ7RDXAVCNFSM54IMCRM2U5DIOJSWCZC7NNSXTN2JONZXKZKDN5WW2ZLOOQ5TCOJRGUZDSOBUGIZA> . You are receiving this because you were mentioned.Message ID: ***@***.***>

[braughtg]

Yes i have been assigned to comment on an issue for my course

Most classes use their own copy of FarmData2 for the course. Your faculty should have given you the URL of the repository for your class.