parth-io / pe

0 stars 0 forks source link

The UG does not mention what is valid syntax for client, reminders, and tags #14

Open parth-io opened 1 year ago

parth-io commented 1 year ago

Some context about the acceptable syntax should be provided in the UG.

For example, the requirement for clients' names is "Names should only contain alphanumeric characters and spaces, and it should not be blank" from the app itself.

But this is not stated in the UG

nus-pe-bot commented 1 year ago

Team's Response

Ref below:

image.png

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: I disagree respectfully that "Names should only contain alphanumeric characters and spaces, and it should not be blank" is a nitty-gritty detail. It is a core aspect about how the inputted names are parsed. For example, if the client's name is "Thiru s/o Bala", I would expect the app to allow this as it is a valid name in use in the world. But the parser blocks it and the UG should mention this constraint.

Some more context on the different name formats that have caused chaos in the world, and thus demonstrate a need for an explicit statement in the UG on what is a permissible name: https://mashable.com/article/airlines-hyphenated-nameshttps://mashable.com/article/airlines-hyphenated-names

What in my opinion construes a nitty-gritty detail are reallly random statments like:

"Names cannot be blank" <-- of course, this is a given

"Names cannot be more than 1000 characters long" <-- 99.99% of the world's population has much shorter names, so this detail does not need to be mentioned in the UG