jerremyng / pe

0 stars 0 forks source link

Overwrite command not working #4

Open jerremyng opened 4 months ago

jerremyng commented 4 months ago

Information

Following instructions from the overwrite error message, I tried to overwrite a person, but it did not work.

Screenshot 2024-04-19 at 4.49.44 PM.png

Steps to reproduce

Enter the commands in the following order to an empty contact list.

1.add n/Taylor Sheesh p/1234567 e/taytay@taylor.com a/Rhode Island

  1. overwrite n/Taylor Sheesh p/999 e/taytay@taylor.com a/Rhode Island

Expected

The phone number listed in the contact list is expected to change.

Actual

Phone number remains unchanged. See screenshot below.

Screenshot 2024-04-19 at 4.50.40 PM.png

soc-se-bot commented 4 months ago

Team's Response

Hi, thanks for pointing this out, I believe that from the usage of this, we should first directly refer to the User Guide in executing the commands. I acknowledge that there is a mistake in the error message, the left out index, but I believe through following the User Guide's prescribed format, this issue shouldn't be a dealbreaker!

Items for the Tester to Verify

:question: Issue type

Team chose [type.FeatureFlaw] Originally [type.FunctionalityBug]

Reason for disagreement: [replace this with your explanation]


## :question: Issue severity Team chose [`severity.Low`] Originally [`severity.Medium`] - [ ] I disagree **Reason for disagreement:** [replace this with your explanation]