nus-cs2103-AY2021S1 / pe-dev-response

0 stars 0 forks source link

Wrong response message #2612

Open nus-pe-bot opened 3 years ago

nus-pe-bot commented 3 years ago

After logging in with sample command edit -l u/nusstu\e1234567 p/testpassword

This was the output. It has no indication it is successful

image.png

However ug claims: Within the red rectangle box, Jarvis will notify you if the login was successful and the logininformation entered.


[original: nus-cs2103-AY2021S1/pe-interim#4472]

wireseo commented 3 years ago

Team's Response

The past tense of "Edited" indicates that the edit was successful, and this is the exact expected behavior of any successful command in our application. In any other cases, there will be a very clear negative indicator that indicates that the edit has failed due to some error.

Duplicate status (if any):

--