search

wanghejin / pe

0 stars 0 forks source link

Overzealous input validation #14

Open wanghejin opened 10 months ago

wanghejin commented 10 months ago

User may want to input two phone nubmers. image.png

nus-pe-bot commented 9 months ago

Team's Response

Hi, I am the dev responsible for this! While yes we acknowledge that this is a potential issue, it is in fact possible to key in two phone numbers (e.g. you can insert p/91234567(home) 46381639(office)). We mention that unless otherwise stated all fields are 1-to-1 in the command format. We do explain how this works in the warning appendix, but we chose not to give an exhaustive list of examples since this would make the UG excessively long. Hence we consider it not in scope.

Items for the Tester to Verify

:question: Issue response

Team chose [response.NotInScope]

Reason for disagreement: [replace this with your explanation]

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