Dethada / pe

0 stars 0 forks source link

Trying to add a person with info `info i/21/05/2024` fails #8

Open Dethada opened 3 months ago

Dethada commented 3 months ago

Bug Description

Trying to add a person with info info i/21/05/2024 fails

Steps to Reproduce

Expected Behavior

The command should succeed.

Screenshots

image.png

nus-pe-bot commented 3 months ago

Team's Response

Rejected as this is the expected behaviour of the app as stated in the UG. The app does not accept multiple identical prefixes, as shown in the error message above

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: What I have reported is not that the app cannot accept multiple identical prefixes but that the information handling is flawed, the intended info to store is "info i/21/05/2024" hence i/info i/21/05/2024 in the add command.

Modifying the original add command to i/info z/21/05/2024 from i/info i/21/05/2024 succeeds, which shows that the error is caused by mishandling of "i/" inside the info parameter.

The original command should succeed as the UG reads:

image.png

Hence the app should be able to handle all inputs users decided to store other than empty values, including those containing " i/".