medajet / pe

0 stars 0 forks source link

UG gives wrong fields for markapplicant #7

Open medajet opened 2 years ago

medajet commented 2 years ago

Fields displays ID when index should be used instead.

image.png

nus-pe-script commented 2 years ago

Team's Response

The format, flags as well as command description correctly write index instead of ID. Furthermore, the applicants do not have an ID. They only have an index, hence it will probably be intuitive for them to use the index, instead of wondering where is the ID. Hence this issue should be of low severity as users will only spot the issue under the "fields" portion. Can be considered a typo. Our error message for the mark applicant command also states to use index.

image.png

Items for the Tester to Verify

:question: Issue severity

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

Reason for disagreement: [replace this with your explanation]