search

ValenciaLim / pe

0 stars 0 forks source link

Arguments does not remove additional whitespaces #3

Open ValenciaLim opened 2 months ago

ValenciaLim commented 2 months ago

add cn/Google n/John Doe p/98765432 e/johnd@example.com a/311, Clementi Ave 2

image.png

nus-pe-script commented 2 months ago

Team's Response

Thank you for your kind suggestion. This feature of removing additional whitespaces for names is not supported currently and it is not within our scope due to existing project priorities and resource constraints. Our team is currently focused on addressing critical bugs and implementing essential features to enhance user experience. However, we have noted your feedback and will consider it for future updates. Thank you for your understanding.

Items for the Tester to Verify

:question: Issue response

Team chose [response.NotInScope]

Reason for disagreement: I feel that this is an overlook on the team's side as this issue could have been easily found and addressed with proper testing and considerations given when they are doing their iP and tP. Additionally, no mention of this issue in their DG and UG.

## :question: Issue type Team chose [`type.FeatureFlaw`] Originally [`type.FunctionalityBug`] - [x] I disagree **Reason for disagreement:** It is a functionality bug as a legitimate user behaviour, in this case, extra whitespaces is not handled. It is not a feature flaw as the add command is a complete feature and solves the stated problem.