search

cleowenxuan / pe

0 stars 0 forks source link

Requires non CLI to trace through the appointments #4

Open cleowenxuan opened 4 months ago

cleowenxuan commented 4 months ago

As mentioned in the textbook, the application should be constrained to typing (CLI) as the interaction method. This is a violation of the constraint, but since it does not mean that the entire feature cannot be used, it is considered as low severity.
video:https://raw.githubusercontent.com/cleowenxuan/pe/main/files/4ad69d1e-fe92-4f5b-ba8b-83720038e7cc.mov

nus-pe-script commented 4 months ago

[IMPORTANT!: Please do not edit or reply to this comment using the GitHub UI. You can respond to it using CATcher during the next phase of the PE]

Team's Response

Double clicking an appointment to trace it is functionally the same as typing “trace [appointment_id]”. As stated on the course website, we are NOT constrained to purely CLI inputs, but rather we are “CLI-first”. We implemented contact tracing through the CLI, and this double-clicking is an additional quality-of-life feature for users.

You can refer to our user guide section on "contact tracing" here.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: [replace this with your reason]