search

alex-setyawan / pe

0 stars 0 forks source link

Unspecified behaviour for adding gender in 'Add' command #3

Open alex-setyawan opened 2 months ago

alex-setyawan commented 2 months ago

The user guide does not specify that when inputting "Prefer not to say" for the Gender field when using the "add command", the app still accepts the input.

Steps to reproduce:

  1. Type "add ic/S9974944F n/John Doe p/91234567 e/johndoe@email.com g/PREFER NOT TO SAY b/11-11-1990 d/Paracetamol | Penicillin i/Infectious Diseases"

  2. Press "enter"

The result is as shown

Screenshot 2024-04-19 162317.png

soc-pe-bot commented 2 months ago

Team's Response

Good catch! Though we believe that it can be of type DocumentationBug as the user guide did not get to specify that entering 'prefer not to say' and excluding g/ from the command are the same thing. The feature is designed to also accept 'prefer not to say' values as an alternative to excluding it .

Items for the Tester to Verify

:question: Issue type

Team chose [type.DocumentationBug] Originally [type.FunctionalityBug]

Reason for disagreement: The phrasing in the user guide is as follows:

image.png

I think this a functionality behaviour as the specified behaviour differs from the actual behaviour in this way:

Specified: M, F or blank Actual: M, F, blank, or any variant of "preferred not to say"