nus-se-bot
commented
2 years ago Team's Response
Forgot to remove from UG after modifications to accepted user input for time field. Accepted
The 'Original' Bug
[The team marked this bug as a duplicate of the following bug]
User not allowed to add All day
For adding an item "All day" was not recognised in the time field, unlike in the UG
[original: nus-cs2113-AY2122S1/pe-interim#2030] [original labels: severity.Medium type.FunctionalityBug]
Their Response to the 'Original' Bug
[This is the team's response to the above 'original' bug]
Previously the time field was just a string. Then I changed it to only accept 24hr format strings but missed out on updating the UG.
Severity.medium since while the flaw does cause inconvenience to users, the product can still be used.
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)
- [ ] I disagree
Reason for disagreement: [replace this with your explanation]
There is no example provided for "All day" items in the UG. This makes it confusing as the reader is not sure where to place "All day" within the code.
The reader may think to place "All day" as ITEM_TIME or as "/All day" to replace "/time".