search

Kaya3842 / pe

0 stars 0 forks source link

App does not support adding of name with s/o (denoting son of) #3

Open Kaya3842 opened 5 months ago

Kaya3842 commented 5 months ago

The app should support adding of name with s/o as it is a valid name.

image.png

soc-pe-bot commented 5 months ago

[IMPORTANT!: Please do not edit or reply to this comment using the GitHub UI. You can respond to it using CATcher during the next phase of the PE]

Team's Response

The error message is correct, and our decision to accept only alphanumeric characters is because / is used in the command format. This is also a problem that does not cause major inconvenience to the user, and can easily be avoided by using "son of" or "daughter of" instead of the abbreviated form, although it is a good suggestion which can be implemented.

It has been specified that the character / cannot be part of the parameters. From the UG: Screenshot 2024-04-20 at 11.56.42 PM.png

While this may be a good feature to have, changing the command format to accommodate the character is out of the scope of v1.4. and it fulfils the second point here from the textbook. Screenshot 2024-04-20 at 11.58.57 PM.png

Items for the Tester to Verify

:question: Issue response

Team chose [response.NotInScope]

Reason for disagreement: [replace this with your reason]