derekjxtan / pe

0 stars 0 forks source link

Room type is tied to room number #10

Open derekjxtan opened 9 months ago

derekjxtan commented 9 months ago

All room types are tied to room numbers, and there is no way to change the number of rooms of each type. I guess for places with fewer than the number of rooms of each type users can work around it, but theres no work around for users with more than the allowed number of rooms. Its a matter of convenience for the first group but unusable for the 2nd group

nus-pe-script commented 9 months ago

Team's Response

The room types are tied to room numbers because in most hotels, room types do not suddenly change (you would need to reconstruct or renovate the room to make it more luxurious), so there is no scenario in which a user would want to change the room types tagged to room numbers.

In the scenario where the room type does change, developers are more than welcome to change the room types tagged to the room numbers.

Items for the Tester to Verify

:question: Issue response

Team chose [response.NotInScope]

Reason for disagreement: The inflexibility imposed by the dev team is quite a big issue as it immediately excludes a number of users that this app is supposed to be catered for. The dev team did address the reason why room tags are fixed for each room but it does not address the issue raised in the bug report. I believe that this is serious design issue since the developers are purposely excluding a subset of their target users for no apparent reason. This inflexibility seems to be added by design based on the dev team's response, which makes it clearly a feature flaw as the feature was not well designed.


## :question: Issue severity Team chose [`severity.VeryLow`] Originally [`severity.Medium`] - [x] I disagree **Reason for disagreement:** I agree with the first point of the dev teams' response. However, I think that what i brought up in my bug report still stands. The current app has strictly 100 rooms of each type. This hard limit should not be imposed as this makes the app unusable for clients with more than 100 rooms of a single type. While this is somewhat acceptable as many hotels are not of this scale, but there is no indication of the product scope being as such anywhere in the user and developer guides. Large hotels can easily hit such numbers, with a more extreme example being MBS which has over 2500 rooms Also, since users are supposed to use the app out of the box, it does not make sense to bring out the argument that "developers are more than welcome to change the room types tagged to the room numbers". If this argument stands, it can applied to justify every feature flaw as not in scope and users can just make corresponding changes.