nus-cs2103-AY2223S2 / pe-dev-response

1 stars 0 forks source link

Target user could be better specified #4173

Open nus-pe-bot opened 1 year ago

nus-pe-bot commented 1 year ago

Type: Design Flaw

Currently,

the target user is for librarians (in general) as in the UG:

image.png

perhaps it could be specified that it is for use in Singapore only (Singapore librarians/libraries) as it is specified that only Singapore phone numbers are valid:

image.png

This may cause inconveniences if someone from a different country wanted to use the application, or if someone in Singapore wanted to use the application, but only has a foreign number.

Perhaps it should be documented as a planned enhancement in the DG if you are planning to enable support for foreign librarians, or make the target user more specific.

Edit: it should be better specified in the DG as well.

image.png


[original: nus-cs2103-AY2223S2/pe-interim#3884] [original labels: severity.Medium type.DocumentationBug]

kynhan commented 1 year ago

Team's Response

Good point raised. However, we believe that the severity of this issue is too high as we will only be marketing this product to Singaporeans and thus, only a few users outside of Singapore will be inconvenienced by this.

Duplicate status (if any):

--