search

weiseng18 / pe

0 stars 0 forks source link

`/` is not allowed in the name field for the add command #2

Open weiseng18 opened 4 days ago

weiseng18 commented 4 days ago

image.png

A name such as John s/o David is not uncommon. As mentioned in the CS2103/T website, this can be classified as FeatureFlaw. Then, since this issue can happen, but the product is still usable, so this could be classified as severity.Medium.

image.png

soc-pe-bot commented 15 hours ago

Team's Response

Screenshot 2024-11-18 at 10.41.07 AM.png

Hi, our team decided to reject this bug as we have clearly indicated it inside one of our planned enhancements (more specifically, point number 6). Moreover, I would argue that a name like John s/o David is rather uncommon and there are many workarounds about it such as writing out in full like "... son of ..." and so on, so even though this can result in rare inconveniences for the user, it is still easy to deal with it and work about it from an end user perspective. So, we believe the more appropriate severity will be severity.Low.

Regardless, this feature flaw is already stated in our planned enhancement, so we decided to reject it, thanks for pointing it out though.

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]