jamesyeap / pe

0 stars 0 forks source link

Class code overly-restrictive #8

Open jamesyeap opened 2 years ago

jamesyeap commented 2 years ago

Suppose I add a student who is in class "101":

Input

add n/Jack Doe p/98765432 e/johnd@example.com a/John Street, Block 123 #01-01 cc/101 s/Negative

Output

Screenshot 2022-04-16 at 3.04.02 PM.png

Comment

Some schools have different formats for class-codes; i.e. some secondary schools can have the class-code 101 mean first class for Secondary 1 level.

As the program's target-school is not explicitly specified, it should not reject class-codes with different formats in order to support whatever format that the school using the application chooses to adopt, else the school would be unable to use the program.

Because of this limitation, I would tag the severity as Medium.

nus-se-bot commented 2 years ago

Team's Response

No details provided by team.

Items for the Tester to Verify

:question: Issue severity

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

Reason for disagreement: ## Reason for disagreement I believe that the severity of this limitation should be kept as Medium, as its impact on the user fits the criteria as defined in the CS2103T textbook (reproduced below) and for the reasons specified in the original bug report,

Screenshot 2022-04-20 at 7.19.54 PM.png