sebin0817 / pe

0 stars 0 forks source link

Valid name not accepted #4

Open sebin0817 opened 2 months ago

sebin0817 commented 2 months ago

Screenshot 2024-04-19 at 4.22.13 PM.png

Screenshot 2024-04-19 at 4.23.27 PM.png

Tested: edit n/Zoé / edit 9 n/Krishnan s/o helloworld Expected: Accepted as a valid name as accented names and 'S/O' or 'D/O' are common Actual: An error was raised

nus-pe-script commented 2 months ago

[IMPORTANT!: Please do not edit or reply to this comment using the GitHub UI. You can respond to it using CATcher during the next phase of the PE]

Team's Response

Thank you for reporting this. However, our team feels that this should be considered a feature flaw instead.

Our UG states:

NAME format: Can only consist of alphanumeric characters and spaces. Extra spaces between values will be processed as a single space. Screenshot 2024-04-21 at 14.04.59.png

This should not be considered as a functionality bug as it is expected behaviour. Furthermore, our DG contains a planned enhancement that addresses this issue:

Screenshot 2024-04-21 at 14.06.13.png

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: [replace this with your reason]


## :question: Issue type Team chose [`type.FeatureFlaw`] Originally [`type.FunctionalityBug`] - [ ] I disagree **Reason for disagreement:** [replace this with your reason]