search

irwynliong / pe

0 stars 0 forks source link

add_interviewer command issue #3

Open irwynliong opened 2 months ago

irwynliong commented 2 months ago

image.png

image.png

Same as add_applicant where i want to add tags for the interviewer but it got overwritten by interviewer tag

nus-pe-script commented 2 months ago

Team's Response

No details provided by team.

The 'Original' Bug

[The team marked this bug as a duplicate of the following bug]

add_applicant command issue

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

[original: nus-cs2103-AY2324S2/pe-interim#3322] [original labels: severity.Medium type.FeatureFlaw]

Their Response to the 'Original' Bug

[This is the team's response to the above 'original' bug]

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 duplicate status

Team chose to mark this issue as a duplicate of another issue (as explained in the Team's response above)

Reason for disagreement: [replace this with your explanation]

## :question: Issue type Team chose [`type.FunctionalityBug`] Originally [`type.FeatureFlaw`] - [ ] I disagree **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.