nus-pe-bot
commented
4 years ago Team's Response
As the attendance GUI panel displays the relevant information according to the listAttendance command, there may be cases where the user deletes a student from a tutorial that s/he is not currently viewing (eg. T01 is being displayed in the attendance list but deleteTutorialStudent was called for T02). Although this may not be the typical case (ie. users will usually be viewing the attendance list of the tutorial they are deleting a student from), we wanted to account for all cases. In the example brought up above, if the suggestion were to be followed, we would be unable to predict if the user wants to continue viewing attendance for T01, or jump to the attendance list of T02, the tutorial they just deleted a student from. Furthermore, if we made the decision to jump to T02, we would be unable to ascertain which week they wish to view the attendance for. As such, to mitigate these uncertainties and allow greater flexibility for the user, we decided to change the attendance view back to default. (Sorry, I am very wordy but this is not paggro hahaha thank you for the feedback!)
Items for the Tester to Verify
:question: Issue response
Team chose [response.Rejected]
- [x] I disagree
Reason for disagreement: Well, I'm not insisting that the view be changed to the tutorial which a student was recently deleted from but rather the view should not change. It shouldn't revert back to the default display
After deleting student from tutorial, existing tutorial attendance for that week and tutorial disappears and the above is shown ^.
Personally I feel that it should just show the new attendance list less the student removed