chiangqinkang / pe

0 stars 0 forks source link

"/" not allowed in tag but not stated in user guide #8

Open chiangqinkang opened 1 week ago

chiangqinkang commented 1 week ago
  1. Run add n/Betsy Crowe s/f r/parent e/betsycrowe@example.com a/Newgate Streetp/12345678 t/new/student
  2. Notice error message "Invalid command format! add: Adds a person to the address book. Parameters: n/NAME s/SEX r/ROLE p/PHONE e/EMAIL a/ADDRESS [t/TAG]... Example: add n/John Doe s/F r/Student p/98765432 e/johnd@example.com a/311, Clementi Ave 2, #02-25 t/friends t/owesMoney"
  3. Notice that user guide does not state that "/" is not allowed in tags.

Screenshot 2024-11-15 at 4.46.20 PM.png

Screenshot 2024-11-15 at 4.45.10 PM.png

soc-se-bot commented 1 week ago

Team's Response

Thank you for your feedback!

The command you’re using is not in the correct format. Specifically, Streetp/12345678 combines the a/ADDRESS and p/PHONE parameters without a space in between. The correct format requires a space before each parameter prefix (e.g., ...[SPACE]p/PHONE[SPACE] ...). The parameters required a space before them, once you add in the space, it works, outputs as shown below

If you try add n/Betsy Crowe s/f r/parent e/betsycrowe@example.com a/Newgate Street p/12345678 t/new/student, with a space between a/Newgate Street and p/12345678 it works.

Outcomes after separating p from Street:

image.png

We hope this explanation provides clarity. Thank you for understanding!

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: [replace this with your explanation]