nus-cs2103-AY2324S1 / pe-dev-response

0 stars 0 forks source link

List-s command executed whenever any schedule is edited #464

Open nus-se-script opened 12 months ago

nus-se-script commented 12 months ago

When the edit-s command is used, the list of schedules returns to display all schedules even after list-s m/0 is used previously to only see all missed schedules.

Steps to reproduce: Have some schedules be missed and others be marked as completed. Then use the list-s m/0 command to look for all schedules marked missed. Then edit a schedule.

Command inputs: list-s edit-s 1 st/2023-09-15T08:58

image.png

image.png


[original: nus-cs2103-AY2324S1/pe-interim#384] [original labels: severity.Low type.FunctionalityBug]

mfjkri commented 12 months ago

Team's Response

Maintaining this behaviour from AB-3 was a design choice.

The most important thing to show the user after an edit schedule command is the edited schedule. Edits made to the schedule might result in the schedule being no longer displayed as part of the filtered schedules (by status, assigned tutor etc).

Users may find this behaviour undesirable as they are unsure whether the edit was successfully made. Clearing the filter and listing all schedules after all edits is the only sure way of ensuring this behaviour (that the edited schedule is shown).

Duplicate status (if any):

--