search

fsgmhoward / CS2113T-pe

0 stars 0 forks source link

Inconsistency in facility type #6

Open fsgmhoward opened 3 years ago

fsgmhoward commented 3 years ago

For listAllLocations, LTs are typed as lecture theater.

However, for search and findFacility, it is typed as lectureTheater without the space in between.

Adding/removing the space in either cases will render the command fail. There should be one consistent spelling for lecture theater.

nus-pe-bot commented 3 years ago

Team's Response

Thank you for pointing out the inconsistency in input format of "lecture theater"

Items for the Tester to Verify

:question: Issue severity

Team chose [severity.VeryLow] Originally [severity.Low]

Reason for disagreement: Very Low is only for typo and cosmetic flaws. I would say this inconsistency is causing inconvenience to users as users will find it very hard to remember which one has the space and which one does not have. Since the difference is only one space, I rated it Low, or I feel it can be at least a 'Medium' as almost all users will feel it inconvenient.