dexter-sim / pe

0 stars 0 forks source link

Unable to link a task to more than 1 exam #7

Open dexter-sim opened 2 years ago

dexter-sim commented 2 years ago

It is quite frequent for a task to be linked to more than 1 exam so the app should allow a task to be linked to more than 1 exam.

image.png

nus-pe-bot commented 2 years ago

Team's Response

Thanks for the bug report. We understand that it might be good for the app to allow a task to be linked to more than 1 exam, as it would increase the flexibility for the user in this application. However, this seems to be an extension of an existing feature of allowing the linkage of one task to an exam.

We feel that this can be considered as Response.NotInScope as per the definition below, as the app is reasonably useful with the current one-to-one linkage of tasks and exams, and rectifying the feature is less important than the work that has been done already. In other words, it seems like a good to have feature, but isn't a must have for the application to work reasonably well.

image

Implementing tasks that can be linked to more than 1 exam would require a lot more work and is non-trivial, as the completion bars for tasks and modules would have to change accordingly to account for this. The completion bars being referred to are the ones in the exam panel below.

image

Furthermore, the deletion and editing of tasks and exams would have to take this into account, given that tasks are linked to exams. For example, when an exam is deleted/edited, all the tasks with this exam should be deleted/edited as well. This implementation is non-trivial and requires comprehensive redesigning of multiple other crucial features (edit/delete) for this purpose. In this case, we feel that this bug is not really in scope and should be categorised as Response.NotInScope.

Items for the Tester to Verify

:question: Issue response

Team chose [response.NotInScope]

Reason for disagreement: [replace this with your explanation]


:question: Issue severity

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

Reason for disagreement: The developer team did not give any response as to why they downgraded the severity from medium to low.

I still feel that the original label of medium severity is apt for this bug report. From a user's perspective, the inability to link one task to more than one exam is bound to cause occasional inconvenience. For example, the lab assignments (task) in CS2030S are important preparation for both practical exam and final exam (exam). The user will have to choose between the exams to link the task to. This could cause a problem where the user may think that they have completed all the tasks linked to an exam when in fact they have not. This is because there could be other incomplete tasks that should be linked to this exam but the user is unable to link each of those tasks to multiple exams.

Therefore, it appears in more than just in very rare situations and causes more than a minor inconvenience. It fits the description of medium severity bug as it is flaw that causes occasional inconvenience to some users, but they can continue using the product.

image.png