search

Justinhoejj / pe

0 stars 0 forks source link

Summary does not account for multiple visits #5

Open Justinhoejj opened 2 years ago

Justinhoejj commented 2 years ago

State of the application with multiple visits upcoming for James image.png

Summary does not show upcoming visits in the next 7 days for James

image.png

I would expect the summary to show 2 missed visit and 6 upcoming in 7 days, accounting for recurring visits not just total number of elderly for the week.

nus-pe-bot commented 2 years ago

Team's Response

The summary screen shows that it is scheduled visits, but it was actually referring to the most recent next visits. This is intended behaviour, but the phrasing was a bit off. Users can still continue to use the app assuming that it only counts one visit for each elderly.

Items for the Tester to Verify

:question: Issue severity

Team chose [severity.Medium] Originally [severity.High]

Reason for disagreement: Attached screen shot from UG which propose that the summary function tracks UPCOMING visits.

image

To re-iterate, the summary function should show upcoming visits as describe in the UG and implied by the phrasing in the UI, if I have scheduled 6 daily visits to a specific elderly for the week, it should show 6 upcoming visits. Otherwise, as a user, I would not be able to manage my visiting schedule since the application underestimates the number of visits I have. While it is true that the user can still save contacts, and arguably that counts as usable, it defeats the intended purpose of the app, a user is better off using a typical contact book rather than SENIOR LOVES that underestimates their visits for the week.

The team masquerades this as an intended behaviour of the application, however instead of rejecting they lowered it to medium which appears to be an acknowledgement of the bug but a failure to acknowledge the bug being severe enough to result in the application being unable to serve the purpose it is designed for.