AlexanderTanJunAn / pe

0 stars 0 forks source link

Cannot checkin after clearing #4

Open AlexanderTanJunAn opened 3 years ago

AlexanderTanJunAn commented 3 years ago

Screenshot 2021-04-16 at 4.58.49 PM.png

Cannot checkin after Clearing, based on UG, it seems to mean clear everything, but program still keeps track of the past id's, which might be confusing.

nus-pe-bot commented 3 years ago

Team's Response

clear command only clears tracking list and not the log file as stated in the FAQ section of the UG:

image.png

Therefore, checking in checkin i/001D n/Tom p/12345678 will lead to an error as the person with the same ID with a different name exists in the log file. However, we recognise that we were not clear about the clear command in the UG. Therefore we accept it as a Low severity of documentation bug.

Items for the Tester to Verify

:question: Issue type

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

Reason for disagreement: Not sure how this will be graded, but this issue was encountered during phase 1 of testing, which is testing the program itself, should this be under DocumentationBug?


:question: Issue severity

Team chose [severity.Low] Originally [severity.Medium]

Reason for disagreement: I believe this is a medium severity as it causes occasional inconvenience.