search

SherwynNg / pe

0 stars 0 forks source link

Allowing users to add bookings with dates that passed #16

Open SherwynNg opened 4 months ago

SherwynNg commented 4 months ago

Steps to reproduce:

  1. input: "book -n CS2103T Consultation -s 2024-03-21 15:00 -e 2024-03-21 16:00 -note TEST"

Expected: I expected it to give me an error message or warning message to display "date has passed" or something similar. Or include in the UG that it allows date that has passed.

Actual: It allowed me to add it under bookings.

Screenshot 2024-04-19 at 5.03.43 PM.png

nus-pe-script commented 4 months ago

Team's Response

Rejected - we allow this for flexibility

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: I feel that if it is allowed, it should be added in the user guide "adding date that has passed for consultation is allowed" or it should display an error message stating that the current date has passed.