search

seaniy / pe

0 stars 0 forks source link

Parameter naming for delete assignment #15

Open seaniy opened 3 years ago

seaniy commented 3 years ago

Perhaps a better parameter name could be given for INDEX as it might not be intuitive for the user on whether it is the module's index or the assignment index. Maybe should be a/ASSIGNMENT_INDEX instead?

image.png

Same applies for deletion of exam.

nus-pe-bot commented 3 years ago

Team's Response

It is clear that the index will be referred to as an assignment index because this is under an assignment command in the UG and right after the assignment prefix "a/" as well.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: [replace this with your explanation]

:question: Issue severity

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

Reason for disagreement: [replace this with your explanation]