Tan-Jin-Waye / pe

0 stars 0 forks source link

Section 4.4.1. Unintuitive fields used for unlink #3

Open Tan-Jin-Waye opened 1 year ago

Tan-Jin-Waye commented 1 year ago

Since each internship is assumed to have at most one person linked, it is unnecessary to have a p/ tag and i/ tag on the unlink command. This is a possible user frustration as they will need to know both the internship index and person index to execute the command even though alternatives are specified in the UG.

Screenshot 2022-11-11 164716.png

nus-pe-script commented 1 year ago

Team's Response

Unlink minimally only requires either 1 of the 2 indexes. Using p/ or i/ is to indicate which list to look for.

So in this case, just doing "unlink i/3" would have worked already.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: [replace this with your explanation]