nus-cs2103-AY2122S1 / pe-dev-response

0 stars 0 forks source link

Unnecessary Case-Sensitive Command for add #2910

Open nus-pe-bot opened 2 years ago

nus-pe-bot commented 2 years ago

User is required to type NA in uppercase when the user does not wish to input any contact number for a particular contact.

Perhaps there is not need for a strict rule for user to type NA in upper-case. A more relaxed rule, such as allowing na in lower-case will be better for fast typer.

image.png


[original: nus-cs2103-AY2122S1/pe-interim#2888] [original labels: severity.Low type.FeatureFlaw]

jianoway commented 2 years ago

Team's Response

The p/ tag is entirely optional to begin with. Including the NA tag was just for congruence with what is displayed in the Academy Directory if the Avenger had itchy fingers and really needed to type in something for the EMPTY p/ field.

Duplicate status (if any):

--