Open nus-pe-bot opened 3 years ago
Reason for rejecting:
Reasons for rejecting:
/schedule filter t/school
. Our current product works correctly, because after running this /schedule filter t/school
command, the Dashboard will indeed show only the upcoming school events, as expected by the user. As expected by the user, the irrelevant events that the user does not want to see will indeed be filtered away./schedule filter t/school
), while the user is on the Dashboard tab, and immediately see the changes synced to the Dashboard tab. So our current product works correctly as expected from the user's point of view./schedule filter t/school
. Nothing happens, nothing in the Dashboard changes. Following your suggestion will result in the wrong product behaviour.Reasons for changing severity from High to Low:
/schedule list
command, which is only a minor inconvenience.Screenshot of User Guide's description of Dashboard Tab:
--
Note from the teaching team: This bug was reported during the Part II (Evaluating Documents) stage of the PE. You may reject this bug if it is not related to the quality of documentation.
Filtering out the schedule causes the dashboard to have the following issue:
It causes the dashboard to have empty upcoming schedules. I do not think that this is an intended behavior, as the list is supposed to provide upcoming schedules. This might be a high feature flaw as the main functionality of the upcoming schedule does not match the behavior itself.
[original: nus-cs2103-AY2122S1/pe-interim#4363] [original labels: type.FeatureFlaw severity.High]