tyx021 / pe Goto Github PK

Expected: Only deletion of vaccinations that cause some appointment to become invalid should be rejected.

Actual: Only 2 of 12 vaccination cleared, while I only have one of them in appointment (Does 2 (Moderna))

Flaw: As a user with external monitor, I wish to view the app easier when I resize the window, however the components will not enlarge as the window

Problem: Phone number can only contain numbers.
There may be some patients from oversea, in this case, they can't add their phone number with country code.

To reproduce: patient detail 0

Expected: Index is not a non-zero unsigned integer.

Actual: Invalid command format

SS:

However, appointment add --p 0 --s 2023-05-01 0700 --e 2023-05-01 0800 --v Dose 1 (Moderna) will display a more appropriate error message

To reproduce: appointment add --p 1 --s 2500-05-01 0700 --e 2500-05-01 0800 --v Dose 2 (Moderna)

Expected: Should reject input for appointment in year 2500

Actual: Input accepted

SS:

The notation is confusing and the meaning of arrow -> is not explained anywhere.

commands: patient add --n Tan Xiao Ming --n abc --p 97643158 --d 1999-04-15 --b A+ --v Dose 1 (Moderna)

If multiple values specified for the same flag, the latest will be accepted. However this is not specified in UG.

To reproduce: Enter some invalid command, e.g. find dose

Expected: Maybe some link to UG to help user familiarizing the app, similar to error invalid command format

Actual: Unknown command

SS:

Reproduce: appointment add --p 5 --s 2023-05-01 0700 --e 2023-05-01 0800 --v Dose 1 (Moderna)

Error message: Patient cannot take the vaccination, user might not understand what does this mean. Assumption should be stated, or displayed in error message

If I have more than 1 patients with the same name , there is no easy way for me to differentiate them, other than noting down which index belongs to which user

Suggestion: Should ask user to specify some unique identifier (e.g. NRIC)

User might be accidentally used clear commands, however there isn't any way to undo the changes.

Not specified in Planned features as well.

There's no Glossary section to explain what are the phrases used in the guide.

e.g. Index can have different meaning depends on the context, but only the constraint is stated.
other example: flag, preamble ...