[Phone input] Phone input should be validated to not exceed unreasonable size

[nus-pe-bot]

Hello!

I do not think this is too big of a deal, but perhaps some input validation constraints for phone number should be good too.

I ran the command edit S1234567B p\00072384724982479828923, which was allowed.

I also feel that despite the Singaporean context, the + symbol should be allowed for international patients. eg. +88 82738492 does not work: edit S1234567B p\+88 89274892

Without space in-between numbers

Thank you! :)

---

_{[original: nus-cs2103-AY2324S2/pe-interim#5262] [original labels: severity.Low type.FeatureFlaw]}

[Jayne1010]

Team's Response

Thanks for pointing this out, while we disagree with the input validation (we wanted to prevent overzealous checking), we do agree that the + symbol might be useful, which was something we did not think about. Good catch!

Duplicate status (if any):

--