search

HenryGan138 / pe

0 stars 0 forks source link

Wrong input format accept #4

Open HenryGan138 opened 4 months ago

HenryGan138 commented 4 months ago

image.png

The program did not call for exemption when I add "/to" without index.

nus-pe-bot commented 4 months ago

[IMPORTANT!: Please do not edit or reply to this comment using the GitHub UI. You can respond to it using CATcher during the next phase of the PE]

Team's Response

The user inadvertently missed some details in the UG that covered the handling of such inputs in the summarise command section

The 'Original' Bug

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

Incorrect command accepted without exception warning

image.png When I input command without following UG, there is no exception warning reported, and the program runs as normal.

[original: nus-cs2113-AY2324S2/pe-interim#2162] [original labels: type.FunctionalityBug severity.Medium]

Their Response to the 'Original' Bug

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

image.png As stated in the last sentence of the UG, "Any extraneous or invalid parameters will be disregarded.", as such this feature is intended

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 reason]

## :question: Issue response Team chose [`response.Rejected`] - [ ] I disagree **Reason for disagreement:** [replace this with your reason]
## :question: Issue severity Team chose [`severity.Medium`] Originally [`severity.Low`] - [ ] I disagree **Reason for disagreement:** [replace this with your reason]