search

nus-cs2103-AY2223S1 / pe-dev-response

0 stars 0 forks source link

Inconsistent UI test cases #2983

Open nus-pe-bot opened 2 years ago

nus-pe-bot 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

[original: nus-cs2103-AY2223S1/pe-interim#2513] [original labels: severity.VeryLow type.DocumentationBug]

lizekai-richard 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.

Duplicate status (if any):

--