search

glenngnng / pe

0 stars 0 forks source link

UG does not specify that list parameters are not case sensitive #4

Open glenngnng opened 1 year ago

glenngnng commented 1 year ago

image.png

in the ug, it is not specified that d/Engineering is case insensitive.

It is possible to still do d/engineering and receive the same results.

image.png

nus-pe-script commented 12 months ago

Team's Response

It is intended to work that way. While I agree that it can be specified, I believe it does not hinder usage at all as in real-life department names should not be case sensitive, e.g. Sales should be the same as sales.

Items for the Tester to Verify

:question: Issue response

Team chose [response.NotInScope]

Reason for disagreement: Furthermore, in the course website, it states that at least one of the bellow needs to be satisfied: The UG specifies it as not supported or coming in a future version. The user cannot attempt to use the missing feature or when the user does so, the software fails gracefully, possibly with a suitable error message i.e., the software should not crash.

But, this is a minor typo that does not satisfy not in scope.