search

shawnnlimm / pe

0 stars 0 forks source link

Cannot differentiate 2 appointments with people of the same name #7

Open shawnnlimm opened 5 months ago

shawnnlimm commented 5 months ago

Description

When 2 people of the same name is added with different appointment times, users cannot differentiate who belongs to which appointment.

Steps to reproduce

  1. Execute add n/hanni pham p/87654321 a/999A t/friend d/tdy 10am-12pm

  2. Execute add n/hanni pham p/87654320 a/999A t/friend d/tdy 12pm-1pm

Expected Behaviour

UI should be able to show details to differentiate between the 2 people

Actual Behaviour

Unable to tell which patient is assigned to which appointment since they have the same name

image.png

Label elaboration

Severity: Low, since it is unlikely to affect a user unless there are people with the exact same name

BugType: FeatureFlaw, since the UI should have more information to distinguish people.

nus-pe-script commented 5 months ago

Team's Response

Thank you for your feedback!

Indeed, the appointment UI could have more information to better differentiate people with the same name, such as displaying the phone number. However, when clicking on a particular “hanni pham” patient card (LHS), we will be able to view their unique appointments.

Hence, we believe this to be a flaw that we will look to improve in future iterations, not a bug, and we designate this issue as NotInScope, following the course website's description of NotInScope as: "a flaw (e.g., a missing feature, a suboptimal design of a feature, a known bug) can be considered NotInScope if rectifying it is less important (based on the value/effort considerations) than the work that has been done already (because it is fine to delay lower priority work until future iterations)." which we believe encompasses this issue.

Items for the Tester to Verify

:question: Issue response

Team chose [response.NotInScope]

Reason for disagreement: [replace this with your explanation]