search

Chiarena / pe

0 stars 0 forks source link

Missing screenshot for marking attendance #10

Open Chiarena opened 2 months ago

Chiarena commented 2 months ago

image.png It could be good to add a screenshot for the attendance panel to show that the absent students have been added to the respective weeks

nus-pe-bot commented 2 months ago

[IMPORTANT!: Please do not edit or reply to this comment using the GitHub UI. You can respond to it using CATcher during the next phase of the PE]

Team's Response

Thank you for the suggestion. However, we intentionally did not include a screenshot of the attendance panel in this section. The attendance panel will only update if the user was previously viewing the attendance panel. If the user was viewing a student's detailed information, the attendance panel would not be updated.

Therefore, it might be confusing to the reader if we were to include a screenshot of the attendance panel in this section as he would then expect the attendance panel to be displayed.

Instead, we have included a screenshot of the attendance panel under the ls command.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: [replace this with your reason]