Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

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

Open
Dethada opened this issue Apr 19, 2024 · 1 comment
Open

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

Dethada opened this issue Apr 19, 2024 · 1 comment

Comments

@Dethada
Copy link
Owner

Dethada commented Apr 19, 2024

Bug Description

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

Steps to Reproduce

  • add cn/Google n/John Doe p/98765432 e/johnd@example.com a/311, Clementi Ave 2, #02-25 tt/121220221400 i/info i/21/05/2024 s/5000 pl/Java t/friends t/owesMoney pri/2

Expected Behavior

The command should succeed.

Screenshots

image.png

@nus-pe-bot
Copy link

nus-pe-bot commented Apr 22, 2024

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

❓ Issue response

Team chose [response.Rejected]

  • I disagree

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/".


Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants