RyanNgWH / pe

0 stars 0 forks source link

Unnecessary seperation of `find`, `patient` and `doctor` commands. #12

Open RyanNgWH opened 2 months ago

RyanNgWH commented 2 months ago

All 3 commands accomplish similar things, with the exception that patient only searches patients, doctor only searches doctors and find searches both. This might not be ideal as the user would have to remember 3 commands instead of one.

An alternative to consider would be to use only one find command, but specify a new parameter to search either the patients, doctors or both.

soc-se-bot commented 2 months ago

Team's Response

Thank you for bringing this up. We thought about it and think that your idea about combining the find, patient and doctor commands to a single find command with additional parameter for searching patient, doctor or both is a good idea. We intend to change it to this in the future.

However, as it current stands, the find, patient and doctor commands are working exactly like how we designed and intended (as per the User Guide and Developer Guide) and is not an unexpected issue. Thus, this is a NotInScope issue.

Items for the Tester to Verify

:question: Issue response

Team chose [response.NotInScope]

Reason for disagreement: [replace this with your explanation]