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

Invalid error message shown when adding assignment but with extra parameters #9

Open
seaniy opened this issue Apr 16, 2021 · 1 comment

Comments

@seaniy
Copy link
Owner

seaniy commented Apr 16, 2021

Command to recreate: add m/daw a/wda by/17/04/2021 2359 n/21

Should be a command format error as opposed to incorrect date format.

image.png

@nus-pe-bot
Copy link

nus-pe-bot commented Apr 20, 2021

Team's Response

No response provided.

The 'Original' Bug

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

Incorrect error message for edit assignment

Entered Command: edit m/Test Module One a/1 on/04/05/2019 1400

Expected output: Wrong error thrown when using on/ instead of by/ or d/ for edit assignment.
Should be error for incorrect command format instead.

image.png


[original: nus-cs2103-AY2021S2/pe-interim#727] [original labels: severity.Medium type.FunctionalityBug]

Their Response to the 'Original' Bug

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

The alphanumeric error is not an error, as the prefix "on" is not expected here, so it is taken as part of the assignment prefix "a/" especially when it is the most recent valid prefix for the command format. The content of the error message is correct and so we reduced the severity of the issue as it does not affect the overall use of the app, especially with the proper use of the user guide.

Items for the Tester to Verify

❓ Issue duplicate status

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

  • I disagree

Reason for disagreement: [replace this with your explanation]


❓ Issue severity

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

  • I disagree

Reason for disagreement: The content of the error message is not correct as it does not reflect back to the user that he/she is using the wrong prefix format.


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

No branches or pull requests

2 participants