Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Authors did not mention the format and constraints for the fields #3557

Open
nus-pe-bot opened this issue Apr 16, 2022 · 1 comment
Open

Comments

@nus-pe-bot
Copy link

nus-pe-bot commented Apr 16, 2022

Screenshot 2022-04-16 at 2.58.43 PM.png

For example, the reader is not informed that phone numbers must be at least 3 digits long and emails must have a name and a domain. The examples, though helpful for regular operations, does not illustrate all these constraints fully


[original: nus-cs2103-AY2122S2/pe-interim#3540] [original labels: severity.Low type.DocumentationBug]
@seanlaiys
Copy link

seanlaiys commented Apr 18, 2022

Team's Response

We appreciate your valid concern about the constraints. The preventive measures in disallowing users of our application to add a person with a 1-2 digit phone number or an email without domain are primarily meant to act as a safety net for typographical errors. We believe that it is a rare case that a user actually intends to input a phone number with 1-2 digit(s) or input an email without domains, as it is extremely rare to have such phone number or email.

Moreover, we did not impose any further constraints on what a phone number or email could be such that additional clarifications are needed. Such constraints could often include restrictions to email domains or restrictions to the number of digits in a phone number specific to any country. While specifically stating the constraints does no harm, our team took the approach to concisely and sufficiently guide readers with the use of concrete examples to improve their understanding of what a phone number and email constitute.

These restrictions are also spelled out very clearly in error messages.
Screenshot 2022-04-18 at 4.09.41 PM.png
Screenshot 2022-04-18 at 4.11.37 PM.png

Duplicate status (if any):

--

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants