nus-cs2103-AY2324S2 / pe-dev-response

0 stars 0 forks source link

Requires non CLI to trace through the appointments #4180

Open nus-se-script opened 4 months ago

nus-se-script 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/nus-cs2103-AY2324S2/pe/master/files/4ad69d1e-fe92-4f5b-ba8b-83720038e7cc.mov


[original: nus-cs2103-AY2324S2/pe-interim#3451] [original labels: severity.Low type.FeatureFlaw]

qinboan commented 4 months ago

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.

Duplicate status (if any):

--