search

domaxi / pe

0 stars 0 forks source link

The program will not throw error even when two different options are set #4

Open domaxi opened 3 years ago

domaxi commented 3 years ago

The program doesn't throw warning even when two different options are set

add-e /t CS2113 /timing 2020-10-30 13:00 /end 2020-10-30 15:00 /repeat weekly /repeat daily

Output:

----------------------------------------------------------------------------------------------------
| Warning! The event is very similar to one already stored. Please ensure the input details are    |
|  correct. The following event is still added:                                                    |
----------------------------------------------------------------------------------------------------
| Event: CS2113                                                                                    |
| Date: 2020-10-30    Start: 13:00    End: 15:00                                                   |
| Reminders: None                                                                                  |
| Repeating: daily (Until: Forever)                                                                |
----------------------------------------------------------------------------------------------------

Screenshot 2020-11-13 at 5.01.59 PM.png

nus-pe-bot commented 3 years ago

Team's Response

Was actually supposed to be documented. Error was that I only updated only list-e documentation to indicate that when repeating the option, only the last one will be read.

Whats in UG for list-e:

image.png

Items for the Tester to Verify

:question: Issue type

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

Reason for disagreement: Although stated, an error message would be better to clarify to the user about the error. Therefore, I disagree that the team changes the bug to a Documentation Bug.