search

irwynliong / pe

0 stars 0 forks source link

add_applicant command issue #2

Open irwynliong opened 2 months ago

irwynliong commented 2 months ago

image.png

image.png

there is a wrong display for incorrect format, where t/ is included but it was not recorded, and the tag that i want to tag i.e "friends" is not recorded

nus-pe-script commented 2 months ago

Team's Response

While we should have updated the error message to not show a tag parameter, we argue that this is "VeryLow" severity at best since it is merely a typo in the error message. The features section for adding applicants and interviewers in the UG explicitly already says adding and customising tags is not currently available. Additionally, both the command summary and example usage for add_applicant and add_interviewer show the right usage, further reinforcing that not amending the error message was an oversight on our part.

Items for the Tester to Verify

:question: Issue type

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

Reason for disagreement: [replace this with your explanation]

## :question: Issue severity Team chose [`severity.VeryLow`] Originally [`severity.Medium`] - [x] I disagree **Reason for disagreement:** Since veryLow only entails UG/UI bugs I would re classify it as Low. As it does cause an issue when using the t/ and not being recorded severity.Low : A flaw that is unlikely to affect normal operations of the product. Appears only in very rare situations and causes a minor inconvenience only.