search

yorklim / pe

0 stars 0 forks source link

add_class: Criteria for module field and tutorial field overly strict #10

Open yorklim opened 5 months ago

yorklim commented 5 months ago

When trying to add a class with lowercase letters in module/tutorial field an error given, as there are no lowercase module codes, the application should just allow the use of lowercase and not be case sensitive when adding those field

image.png

image.png

Steps to reproduce:

  1. use "/add_class module/cs1231 tutorial/t01 description/asdsa size 1"
  2. use "/add_class module/CS1231 tutorial/t01 description/asdsa size 1"
nus-pe-script commented 5 months ago

Team's Response

image.png

image.png

Intended behaviour, as shown in the UG.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: This was reported as a feature flaw, not a functionality bug, stating that it is an intended behavior and the letter must be a capital letter does not justify why this was done in this way. In the real NUS system, t09 and T09 mean the same thing, thus it should follow that. Look at case sensitivity under Feature Flaw.