search

jimseah-0116 / pe

0 stars 0 forks source link

Tag error message not correct (in app) #5

Open jimseah-0116 opened 2 months ago

jimseah-0116 commented 2 months ago

Although mentioned in the UG, the app does not state that tags cannot contain spaces. The error message should be updated.

Command used: "add n|guy e|em@il p|1234 a|12345 t|he llo"

image.png

soc-se-bot commented 2 months ago

Team's Response

Thank you for your report. We agree that the app's error message is inconsistent with the UG.

However, this is not an issue as spaces are not alphanumeric. This error message encompass all requirements for tags.

image.png

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: I feel like NotInScope would be more appropriate here. Without getting into the definition, my initial impression of "alphanumeric" is that the tag allows alphabets, numbers, and spaces. This is definitely subjective and I'm not sure about how others interpret "alphanumeric", but personally, my group added this specification that space is not allowed in our UG as well, for our alphanumeric tags.

So while the error message is technically "sufficient" and not necessary at this stage, adding that "space is not allowed" in the app's error message is a simple change that can be easily done and will definitely add clarity.