nus-cs2103-AY2324S1 / pe-dev-response

0 stars 0 forks source link

Able to add classes that are over #987

Open nus-se-script opened 12 months ago

nus-se-script commented 12 months ago

The application allowed to add of classes or schedules that were over. This may not serve the purpose of the apps as it is to solve scheduling conflicts. Suggest further restricting the date and time for adding the schedule.

Steps to reproduce: 1: Enter the command: add-s 1 st/2023-11-01T09:00 et/2023-11-01T11:00

The below screenshot showed that the application allow to add a class that is already over

image.png


[original: nus-cs2103-AY2324S1/pe-interim#825] [original labels: type.FeatureFlaw severity.Medium]

mfjkri commented 12 months ago

Team's Response

The purpose of the app is also to track schedules.

TutorConnect is an address book made for tuition centre coordinators to easily track and schedule tuition classes.

A tuition centre coordinator might want to track classes even if they have happened in the past. As such, we do not restrict the date time of new schedules and it does not hinder users from using the application.

Additionally, overzealous input validation (the opposite of this) is considered a feature flaw bug.

Screenshot 2023-11-19 at 11.57.47 PM.png

Duplicate status (if any):

--