jibtaf / pe

0 stars 0 forks source link

Unnecessarily complicated command formats for edit-appt command #4

Open jibtaf opened 10 months ago

jibtaf commented 10 months ago

The format for this command is: edit-appt APPOINTMENT_INDEX pi/patient-index [d/DATE] [t/TIME]. In my opinion, the pi/patient-index does not seem to be necessary since the users should perform this command after doing list-appointments patient-index, then it does not add more value to probably edit another patient's appointment here.

soc-se-bot commented 10 months ago

Team's Response

This issue is marked as duplicate as this also mentions the design of pi/PATIENT_INDEX.

The 'Original' Bug

[The team marked this bug as a duplicate of the following bug]

Unnecessarily complicated command formats for add-prescription, edit-prescription and delete-prescription commands

The format for the commands are: add-prescription APPOINTMENT_INDEX pi/PATIENT_INDEX mn/MEDICATION_NAME..., edit-prescription APPOINTMENT_INDEX pi/PATIENT_INDEX mn/MEDICATION_NAME... and delete-prescription APPOINTMENT_INDEX pi/PATIENT_INDEX. In my opinion, the pi/patient-index does not seem to be necessary since the users should perform this command after doing list-appointments patient-index, then it does not add more value to probably add/edit/delete another patient's prescriptions here.


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

Their Response to the 'Original' Bug

[This is the team's response to the above 'original' bug]

Thank you for the issue.

Just to note that there are duplicate issues as all the issues mention the design of pi/PATIENT_INDEX

Items for the Tester to Verify

:question: Issue duplicate status

Team chose to mark this issue as a duplicate of another issue (as explained in the Team's response above)

Reason for disagreement: [replace this with your explanation]