yhtminceraft1010x / pe Goto Github PK

Given that there's only 2 possible vaccination statuses, typing out vaccinated and unvaccinated in full may be too time-consuming for a fast-typist. Perhaps it can be shortened to "Vac", "Unvac" etc

Here!

Both add and edit commands use different names for the phone number parameter. While there is not much ambiguity, it can be quite jarring when it comes to consistency.

The colours for the dates are similar to the colours of each appointment. It might be assumed that Sunday 12 December 2021 appointment is for David Li rather than John Doe.

Developer Guide links:

Clicking on Ui.java:

Clicking on MainWindow.fxml:

The links I tested went back to the original addressbook3 links. It is likely that all other github links in the DG end up directing the reader to the original addressbook3.

Command used: add A1234567X n/John Doe p/98765432 e/[email protected] a/John
street, block 123, #1-01 s/vaccinated m/peanut allergy r/RVRC (Missing faculty)

On the application:

On User Guide:

The faculty input is different. The error message shows f/SCI while the user guide shows f/COM.

Based on this list, it may be unclear which faculty should Nursing and Pharmacy students be put into. It can be argued that vaccination is more critical to these 2 groups of students. Perhaps a table/footnote that shows this or indicating "Please refer to your matriculation record for your faculty" may help.

Command used:

add A7654321J n/Betsy Crowe f/ENG p/91119222 e/[email protected]
a/212 Orchard Road, #18-08 s/unvaccinated m/nose lift surgery in 2012

From User Guide:

Result:

If copy and pasted, the a/ prefix gets fused with the input for e/ prefix, resulting in an invalid command format error. I believe that most new users would copy and paste the commands from the user guide to try out the product, thus the medium severity rating.

I'm not sure how the medical history is a subpoint under Who wants to be vaccinated since Who wants has an implication of patient's choice. I think a better way would be to change Who wants to be vaccinated to Who can be vaccinated or to simply make the medical history point its own point.

Application:

User Guide:

I think it would be helpful to have an ability to add multiple appointments for the same student, given that both vaccination appointments can be made in advance (since the other shot is just 3-4 weeks apart) and there could also be appointments for covid testing and checkups (which was mentioned in the user guide).

It appears that the Storage component title is stuck inside the NOTE box. It is also italicised unlike the other titles describing each component.

Even though the User Guide describes how to use the various inputs, it might not be intuitive as some may assume that stats all shows the percentages across the whole NUS, which is the current function of stats NUS, rather than the breakdown of percentages.

A preferred option would be stats breakdownAll to show the breakdown across the faculty and student hostels.

I believe that the deletion by matriculation number can be inconvenient for users of this app. I think most would prefer to delete by index of the record list since it is quicker.

I think that leaving the input formats at the bottom can be confusing for new users since they may read the user guide in a top down manner and see the add command first without knowing the correct format for each input.

While I understand that there is a hyperlink to input formats, I would prefer to put the input format at the top so that new users who read the user guide in a top-down manner can get a glimpse at the input format first and may have more familiarity with the add command.