nus-cs2113-AY2223S2 / pe-dev-response

0 stars 0 forks source link

Failure to notify error of usage of -r switch #785

Open nus-pe-bot opened 1 year ago

nus-pe-bot commented 1 year ago

Following the documentation in the UG,

image.png

I assumed a valid command to be add -e Career Fair -st 14:00 -sd 2023/02/10 -ed 2023/02/11 -r 3 D/ 2W. However I was not notified that I had used an incorrect format.

Steps to reproduce: add -e Career Fair -st 14:00 -sd 2023/02/10 -ed 2023/02/11 -r 3 D/ 2W

Expected output: The event to be added with recurring time interval of 3 days every 2 weeks.

Actual output:

add -e Career Fair -st 14:00 -sd 2023/02/10 -ed 2023/02/11 -r 3 D/ 2W
____________________________________________________________
(since no specific ending time information is given, ending date is omitted)
____________________________________________________________
Event successfully added:

    > [E] Career Fair (2023/02/10 14:00 | recurring, time interval: 3 )

[original: nus-cs2113-AY2223S2/pe-interim#1058] [original labels: severity.Medium type.FeatureFlaw]

kyrixn commented 1 year ago

Team's Response

Similar to issue 2459, we admit that this is caused by unclear instructions in UG. However, instead of the brief instruction of how to use this command that the tester screenshots, The team has also provided an example of the command being used which is explained it better on the line that is below this:

image.png

Duplicate status (if any):

--