ChinSekYi / pe

0 stars 0 forks source link

Complicated command name `addEcName` #5

Open ChinSekYi opened 1 week ago

ChinSekYi commented 1 week ago

Users have to use caps twice just to use the addEcName command. This is not suitable for fast typers as the command is error prone.

Suggestion: parse addEcName to lowercase in logic, so that users who dont use caps lock eg addecname, can still use the command.

Same can be applied to other commands like addEcNumber and addExamScore

soc-pe-bot commented 4 days ago

Team's Response

Justification for not in scope: According to the guidelines, what was provided was a suggestion. Furthermore, the work done to the implement the feature for adding emergency contact name was more important than fixing this minor bug. Moreover, the UG clearly states the command format, as such, there would not be any issues if the users followed the correct command as stated.

image.png

image.png

Items for the Tester to Verify

:question: Issue response

Team chose [response.NotInScope]

Reason for disagreement: [replace this with your explanation]