alyssa-savier / ped Goto Github PK

The application window is a bit large and in minimised view still takes up a majority of my window screen, perhaps it would be good to have a smaller size for this as it can get a bit overwhelming, and it is not as easy to use the app side by side with another app, for example documents about patients.

For most scheduler apps, overlapping times are allowed and I don't think it is necessary to prevent overlapping times to be entered in. Some people may just want to have multiple activities recorded or documented in the app then prioritise later on which activity to attend.

While it is understandable that you assign unique identifiers using PID which is automatically made for the patient, it seems like a case where the same patient is re-entered into the system is not handled, and they are handled as an entirely new patient even though every other field is exactly the same. Perhaps we can use a separate unique identifier that the patient can provide, such as NRIC or Passport number to prevent this.

This feedback (row turning blue) suggests that something had happened, but there is not really anything that happened in the app. Perhaps this could lead to viewing a patient, or perhaps just omit this feature entirely as it can be misleading

It is not stated in the UG how the app handles multiple inputs for a certain field, for example having two phone number inputs, p/123132 p/234353, and instead it takes the last input alone.

It seems that it is not really possible to toggle between the doctors and patients window through the GUI aside from typing list t/<patient/doctor> though the UI design seems to suggest that you can click on the side bar to toggle between it.

The columns are very close to each other in the GUI so it is a bit difficult. Also, the dates are written in a bit of an unintuitive way so perhaps a separate way of presenting the dates might be good, eg 10 May 2021, 3pm.

Scroll speed of the list is a bit high which may be difficult to find patients via scrolling

When entering multiple inputs that are wrong, the error message returned only handles the first parameter with the wrong input.

On the gui it is stated as activities but the cli wants to read activity