nus-cs2103-AY2324S1 / pe-dev-response

0 stars 0 forks source link

Modifying specialization of a doctor is inconvenient #271

Open nus-se-script opened 10 months ago

nus-se-script commented 10 months ago

Lets say I have a doctor that specialize in heart and brain but I want to add a specialization of leg. To do so I will need to do spec 4 heart, brain, leg but if the doctor has a long list of specializations I would need to type out the whole list again which can be error prone and inconvenient. Also, as there are many parts of an area of specialization, it is also inconvenient that brackets () are not allowed. It would be nice to have for example spec 4 brain(front), brain(back)


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

leezhanpeng commented 10 months ago

Team's Response

Thank you for the suggestion tester! However, we designed specialisations based on real-world context. It is very rare for established doctors in a hospital to gain additional specialisations throughout their tenure, as such we do not expect further modifications of their specialisations after it has been set for a while. Additionally, allowing for special characters like brackets does make the command harder to parse, which is a valid reason for the lack of such flexibility as suggested under Guidelines for the dev team to follow when triaging PE bugs in PE Preparation, Restrictions:

image.png

As changes or additions to specialisations are not anticipated once a doctor has been on the contact list for a considerable period, we do not expect any regular inconvenience for users.

Duplicate status (if any):

--