search

mahadhir247 / pe

0 stars 0 forks source link

Missing implementation of DeletePatient in DG #7

Open mahadhir247 opened 4 months ago

mahadhir247 commented 4 months ago
soc-pe-bot commented 4 months ago

Team's Response

Hello, thank you for raising this issue.

Based on the requirements defined by the teaching team previously:

Screenshot 2024-04-21 at 1.13.04 AM.png

There is no need to document every feature.

The team decided that describing the addPatient command prepares any developer reading the guide with sufficient context and knowledge of how things work. The knowledge from the addPatient implementation is directly transferable to that of deletePatient. There is minimal value in including the implementation guide for both deletePatient and addPatient.

Furthermore, if there were any doubts, the sequence diagram for deletePatient is also available in the DG.

image.png

Hope this clarifies.

Response rejected.

In any case, the severity was dropped to severity.Low as at best, it is a minor inconvenience.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: [replace this with your explanation]

## :question: Issue severity Team chose [`severity.Low`] Originally [`severity.Medium`] - [ ] I disagree **Reason for disagreement:** [replace this with your explanation]