wz-chan / pe

0 stars 0 forks source link

Wrong error message for assignptg command #1

Open wz-chan opened 3 years ago

wz-chan commented 3 years ago

Enter "assignptg" to get the error message that is shown in the screenshot below image.png

In the error message, "Parameters: n/PERSON Nameg/GROUP NAME " should be "Parameters: n/PERSON Name g/GROUP NAME"

Not including the space between "n/PERSON Name" and "g/GROUP NAME" will cause an error.

nus-pe-bot commented 3 years ago

Team's Response

Noted. The issue here is that the spacing is unintentionally omitted. However, it should be clear to the user that the space is required.

The 'Original' Bug

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

Inconsistent error message in assignptg invalid command

When type an invalid 'assignptg' command, the error message displayed is not inconsistent with the UG.

what mentioned in the user guid is "n/PERSON_NAME g/GROUP_NAME", however the app displayed "n/PERSON Nameg/GROUP NAME", the space between name and g is missing.

Screenshot 2021-04-16 at 7.29.43 PM.png


[original: nus-tic4002-AY2021S2/pe-interim#12] [original labels: severity.Low type.FeatureFlaw]

Their Response to the 'Original' Bug

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

Hi

Noted with thanks.

Have downgraded the severity. It is just the accidental omission of one space.

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 type

Team chose [type.FeatureFlaw] Originally [type.FunctionalityBug]

Reason for disagreement: This is a bug with the functionality of assignptg command; not a missing functionality that made the command less useful.