nus-cs2103-AY2425S1 / pe-dev-response

0 stars 0 forks source link

Name constraints gap #1567

Open nus-se-script opened 6 days ago

nus-se-script commented 6 days ago

States "S/O" or "D/O" is accepted,


[original: nus-cs2103-AY2425S1/pe-interim#2158] [original labels: severity.Low type.DocumentationBug]

yu-sutong commented 6 days ago

Team's Response

Screenshot_1727.png It is stated that not every nitty-gritty details need to be included. For users putting in "S/O" or "D/O", they are expected to know how "S/O" and "D/O" roughly work in names, or at least be able to refer to the customer's name when inputting the name. Choosing to put an example for all these specific types of names (e.g. S/O, -, ') may end up resulting in a lot of examples due to the large range of types of names. Hence, no example was included for such specific cases.

Screenshot 2024-11-17 at 2.02.25 PM.png Users can interpret from our UG how S/O and D/O can be used and should be in upper case as per how "S/O" and "D/O" normally operates in real-world scenario, which is naturally capitalized. (also, as stated earlier, they are expected to know how "S/O" and "D/O" roughly works)

Hence, this response is rejected.

Duplicate status (if any):

--