NumberInput: add option to allow only positive integers

[jrcastro2]

• closes https://github.com/zenodo/who-rdm/issues/77

[kpsherva]

I think this is bypassing validation (backend or frontend) in an unorthodox way, without providing any feedback to the user on incorrect input. I think it might be confusing for the user
why not using the min html attribute ?
<input type="number" min="0">
or
<input type="number" step="1" pattern="\d+"> this one should give feedback when you try to type -
this approach can be used without adding a new prop the html attributes should be automatically added by react to the output element

[jrcastro2]

Not sure I understand the first part, the user gets the same feedback as when he tries to type any letter, when he types it, it will not appear in the field.
I agree that we should align the validations between the backend and frontend but it's not correct at the moment either, as in the frontend it's allowed to introduce +- sign and when doing this it will not save the number (although it's displayed in the UI). Here an example of this:

Peek.2023-09-12.18-06.mp4

By doing this change it actually aligns a bit more with the backend validation.

I agree that ideally we also want to remove the + and - sign from the backend validation, but I also think we could tackle that in the future.

I also tried your suggestions without any luck, maybe I am doing something wrong:

[jrcastro2]

This is "not needed" anymore, however there is an issue with the InputNumber where you can type +-123 and will be evaluated as a string and since the input type is number it will not be submitted but it will appear filled in the field.