YH-15 / pe

0 stars 0 forks source link

Phone number length not limited #3

Open YH-15 opened 1 year ago

YH-15 commented 1 year ago

The length of phone number could be limited to a reasonable range. I notice that we have a lower bound 3, but it would be better if we could add a upper bound.

The issue could be replicated by using the following command:
addperson n/Ivan p/12151318135131861681531813184864415313555555555555351351862181258 e/1271755549@qq.com a/pasir panjang rd

nus-pe-bot commented 1 year ago

Team's Response

It has been mentioned in the textbook that the length of the input should not matter.

Items for the Tester to Verify

:question: Issue response

Team chose [response.NotInScope]

[ ] I disagree

Reason for disagreement: [replace this with your explanation]