Missing implementation of EditPatient in DG

[mahadhir247]

• DG is missing implementation of editPatient command
• Since it seems the implementation of editPatient is non-trivial, it should be included in the DG
• editAppt is also included in DG, so it is abit unusual to omit editPatient

[soc-pe-bot]

Team's Response

Hello, thank you for raising this issue.

As mentioned in the bug report, the implementation logic for editAppt is present. The general implementation logic for editPatient, at the structural level as covered in the DG, is similar to that of editAppt.

Based on the requirements defined by the teaching team previously:

There is no need to document every feature. The team decided that given the similarities of editAppt and editPatient, developers that understand how editAppt works will find that editPatient works similarly. In fact, editPatient is simpler than editAppt. There is not much value in including the implementation of editPatient on top of editAppt. The learning curve is not steep.

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]

• [ ] I disagree

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]

---