nus-cs2103-AY2324S1 / pe-dev-response

0 stars 0 forks source link

There is no command to edit doctor #5948

Open nus-se-script opened 1 year ago

nus-se-script commented 1 year ago

If I wrongly add a doctor, the only way to fix it is to delete the doctor and add a new one, which is inconvient.


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

Bryan-Goh commented 1 year ago

Team's Response

This feature flaw was already specified in the ug as one of the warnings, and there is already a section in the dg for future features for it. The reason why this feature is implemented as a future feature is because :

  1. The effort required to implement this feature to the current state of the product is way too high as you not only need to consider editing the doctor but you also need the impact editing a doctor's detail has on other feature such as appiintments. This will cause a lot of bugs that is going to take a lot of effort and time to fix and test.
  2. This feature is not strictly required for this product to function as you can just delete a doctor and add another doctor as mentioned in the UG. The current implementation already took a lot of effor and time to implement without a lot of major bugs due to the relationship between patient doctor and appointment. This means that changing one of those aspects can potentially break the software. It took a lot of effort to implement add and delete doctor while not breaking any of the other commands. Adding the edit doctor feature would then further complicate things even more than one can realize and can have a lot of negative effects to the software if done incorrectly. The WARNING in the UG: Screenshot 2023-11-20 at 15.48.38.png The future features in the DG: Screenshot 2023-11-20 at 15.49.18.png Therefore we have fulfilled the conditions for not in scope

From the reasons mentioned above we have fulfilled the condition in order for this feature flaw to be considered NotInScope from the cs2103 website.

Duplicate status (if any):

--