Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Inconsistent error message in assignptg invalid command #107

Open
nus-pe-bot opened this issue Apr 20, 2021 · 0 comments
Open

Inconsistent error message in assignptg invalid command #107

nus-pe-bot opened this issue Apr 20, 2021 · 0 comments

Comments

@nus-pe-bot
Copy link

[destination: nus-tic4002-AY2021S2/pe-moderation assignee: boydanderson labels: response.Accepted severity.VeryLow team.3 tutorial.TIC4002-F18 type.FeatureFlaw pending.0]

Summary of items to check:

Nothing to check :-)



Issue Description

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: Yiheng0410/pe#2]

The team marked the following 2 bug reports as 'duplicates' of this bug
(in case those details are useful when estimating the severity of this bug) -- click to expand:

assignptg invalid command format

Command:
assignptg

Notice the recommended command is missing a space.

image.png

Expected recommended command to be what being describe in the UG.

image.png


[original: nus-tic4002-AY2021S2/pe-interim#40] [original labels: severity.VeryLow type.FunctionalityBug]

Wrong error message for assignptg command

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.


[original: nus-tic4002-AY2021S2/pe-interim#22] [original labels: severity.VeryLow type.FunctionalityBug]

Team's Response

Hi

Noted with thanks.

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

To Check [0 Items]

Nothing to check :-)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant