search

CeereeC / pe

0 stars 0 forks source link

Inconsistent UI test cases #5

Open CeereeC opened 2 years ago

CeereeC commented 2 years ago

Under 4.3.3. Adding a student or teaching assistant:

You added John Doe as a TA for CS1101S.

Screenshot 2022-11-11 at 4.35.39 PM.png

However in the next command for editstu, John Doe is now TA for CS2040S. Might've been better to standardize from top to bottom

Screenshot 2022-11-11 at 4.34.02 PM.png

nus-pe-bot commented 2 years ago

Team's Response

Thanks for your issue, but the purpose of examples is to illustrate how the command works to users. We don't think such inconsistency will hinder users.

Additionally, in the User Guide, it was clearly stated what the issue is and how the command helps to rectify it.

image.png

I do not think that such an inconsistency would hinder users since these commands are specified under different sub sections, with a clear preface of the issue at hand and very clear labelling to indicate what has changed after the command is executed.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: [replace this with your explanation]