search

rageqqq / pe

0 stars 0 forks source link

Not clear whether LESSON_NAME or [LESSON_NAME] #1

Open rageqqq opened 3 years ago

rageqqq commented 3 years ago

image.png

It is not clear whether you are supposed to type LESSON_NAME as part of the command or LESSON_NAME is a variable as LESSON_NAME is not in square brackets like the others

nus-pe-bot commented 3 years ago

Team's Response

Unlike the other fields, lesson name here is compulsory, hence it does not have the square brackets. More information can be found in the user guide.

The 'Original' Bug

[The team marked this bug as a duplicate of the following bug]

Unclear instructions about add task command

image.png

Similar to the add lsn command, it might be confusing to some users as to whether they have to enter the NAME and Date portion as NAME ;; DD-MM-YYYY or enter them as variables (Do homework ;; 14-09-1222). Perhaps using different brackets ( ie. { vs [ ) could make it easier for the user to differentiate in both the UG and the app itself

[original: nus-cs2113-AY2021S2/pe-interim#952] [original labels: severity.High type.FunctionalityBug]

Their Response to the 'Original' Bug

[This is the team's response to the above 'original' bug]

We believe the regular user will understand that entering "NAME ;; DD-MM-YYYY" is the format, and does not represent the actual input value itself as the labels for the fields are in UPPERCASE.

The UG does make the difference very clear. However, perhaps the usage of different brackets may be a suggestion, though it is not in the scope of this PE. Also, the suggestion seem to be for a separate issue.

We disagree that this is a high severity functionality bug.

add task.png

Items for the Tester to Verify

:question: Issue duplicate status

Team chose to mark this issue as a duplicate of another issue (as explained in the Team's response above)

Reason for disagreement: [replace this with your explanation]

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: image.png

UG indicates that this is merely a tip rather than something the user should take note of. I would think that this is a very important point since it is something that cannot be left out. Furthermore it would be easier to differentiate between something that cannot be left out and something that can be skipped by using different brackets which was practiced in the application but not in the UG.

:question: Issue severity

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

Reason for disagreement: Think it should be medium