search

chewbum / pe

0 stars 0 forks source link

Potential Improvement for prefix naming of phone number #1

Open chewbum opened 6 months ago

chewbum commented 6 months ago

Description: There could have been better explanation of the prefixes and place it before they are introduced in the Handling ClientCare error messages section. It was a bit confusing and non-intuitive for a phone number to have a prefix starting with "c/". It was only mentioned right after this section under adding a client.

image.png

soc-pe-bot commented 6 months ago

Team's Response

"1. In terms of documentation bug, ""c/"" is consistent with our application behaviour.

  1. ""c/"" seems reasonable since it is our preference to name it so as it stand for contact number. Also, ""p/"" stands for priority and we cannot have duplicated tags.
  2. Directly above this section, there is a hyperlink to the Command Summary to all the prefixes for the user to explore."

    Items for the Tester to Verify

    :question: Issue response

Team chose [response.Rejected]

Reason for disagreement: I believe that there could still have been a better choice of prefix as the letter c has nothing to do with phone number and it isnt intuitive, however, since this is considered a small enhancement, that was why I declared it as severity very low.