search

derekjxtan / pe

0 stars 0 forks source link

Room statistics window not very helpful #12

Open derekjxtan opened 9 months ago

derekjxtan commented 9 months ago

The room statistics window only shows how many rooms are occupied based on the whole list of bookings which isnt helpful. Would be better to show occupied rooms at a certain period of time.

nus-pe-bot commented 9 months ago

Team's Response

No details provided by team.

The 'Original' Bug

[The team marked this bug as a duplicate of the following bug]

Usefulness of Room Statistics

The current implementation of room statistics shows the breakdown of occupied rooms. This is not very useful as it does not take into account the date of booking into consideration. All 500 rooms may be booked but the user will not be able to tell if the capacity of the hotel is full at any given point in time.

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

Their Response to the 'Original' Bug

[This is the team's response to the above 'original' bug]

  • Currently, our application does not take into consideration the time of the system (ie: your laptop), so there is no way to check if the booking has passed or not. This is out of scope.

  • The app is meant to be used by hotel receptionists when a guest walks in (after all, there is no online version of this app), and when a guest walks into a hotel they would almost certainly book a room immediately , so the booking period will match the day the booking is made.

  • Even if somehow in some scenarios the booking is made and it is in the future, once again the app does not take into consideration the time of the system, so it is out of scope.

    Items for the Tester to Verify

    :question: Issue duplicate status

Team chose to mark this issue as a duplicate of another issue (as explained in the Team's response above)

Reason for disagreement: [replace this with your explanation]

## :question: Issue response Team chose [`response.NotInScope`] - [ ] I disagree **Reason for disagreement:** [replace this with your explanation]
## :question: Issue severity Team chose [`severity.VeryLow`] Originally [`severity.Low`] - [ ] I disagree **Reason for disagreement:** [replace this with your explanation]