yeohhq / pe

0 stars 0 forks source link

Command: 'addlsn' week number display suggestion #1

Open yeohhq opened 3 years ago

yeohhq commented 3 years ago

This issue reports a feature suggestion for the 'addlsn' command.

Input: addlsn grp/g04 lsn/00-1

Output: Screenshot 2020-11-13 at 4.25.21 PM.png

Rationale: I believe that the UG uses 'XX-Y' seen below, which the application shows the correct error message, but '00' would probably be better to translate into just '0', or should not be accepted if lessons should start naming from index 1.

Screenshot 2020-11-13 at 4.23.40 PM.png

nus-se-bot commented 3 years ago

Team's Response

From the website,

image.png

It is a feature flaw if it is less useful to the user.

The product works as expected and is equally useful to the user whether it is represented as 00 or 0

Week 0 is a valid input because NUS has a week 0.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: [replace this with your explanation]