search

ruishanteo / pe

0 stars 0 forks source link

No mention of input constraints #2

Open ruishanteo opened 9 months ago

ruishanteo commented 9 months ago

There is no mention of input constraints in the UG. In the adding person command.

Some of which are names, phone numbers, etc. The unique names property was also not mentioned in UG.

Screenshot 2023-11-17 at 4.16.53 PM.png

nus-pe-script commented 9 months ago

Team's Response

Hi, thank you for pointing out that our UG did not explicitly state the restrictions. However, based on the "Guidelines for the dev team to follow when triaging PE bugs" under PE Preparation, Restrictions, it is stated that this is fine as long as users are informed through error messages:

image.png

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: [replace this with your explanation]

## :question: Issue severity Team chose [`severity.Low`] Originally [`severity.Medium`] - [ ] I disagree **Reason for disagreement:** [replace this with your explanation]