Howlong11 / pe

0 stars 0 forks source link

Reservations for the past are allowed #5

Open Howlong11 opened 2 months ago

Howlong11 commented 2 months ago

image.png

Reservations for the past are allowed. However, if they are already expired, they should not be allowed to be added as new reservations.

nus-se-script commented 2 months ago

Team's Response

This is the expected behavior of the app.

Even if the user adds a reservation of the past, the address book will correctly detect that the reservation has expired, and the expiry tag will correctly be shown.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: If the reservation of the past is added, it should not be allowed to be added, and not just showing an expiry tag. In reality, if a customer calls for reservation on the date that is already passed, no restaurant will be adding the reservation to the past calender.


## :question: Issue severity Team chose [`severity.Low`] Originally [`severity.Medium`] - [x] I disagree **Reason for disagreement:** If reservations of past dates are allowed to added, it may confuse to the restaurant owner whether the reservation is actually expired, or he just made a typo and added an incorrect reservation.