nus-pe-bot
commented
4 years ago Team's Response
We implemented the features (e.g. PatientMap, PatientRecordMap, AppointmentMap) using that particular data structure, so we believe it belongs to the implementation section. Furthermore, we believe that the usage of HashMap in our application is a relatively big feature.
Items for the Tester to Verify
:question: Issue response
Team chose [response.Rejected]
- [ ] I disagree
Reason for disagreement: [replace this with your reason]
:question: Issue severity
Team chose [severity.VeryLow]
Originally [severity.Low]
- [ ] I disagree
Reason for disagreement: [replace this with your reason]
While the implementation section is supposed to be describing how certain features are implemented, I feel that its quite weird to mention the data structure of the feature as one of the subsections (4.1), as the rest of the subsections (4.2 - 4.9) are features and how they have been implemented. E.g. "Add Details Feature", "List Feature" etc are all features while "Data Structure" is not exactly a feature.
Perhaps the data structure section could be put into the implementation subsections of each of the relevant features, instead of being a subsection by itself.