Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[PE-D] Attendance Feature is incomplete #219

Closed
nus-se-bot opened this issue Oct 30, 2021 · 2 comments · Fixed by #223
Closed

[PE-D] Attendance Feature is incomplete #219

nus-se-bot opened this issue Oct 30, 2021 · 2 comments · Fixed by #223
Assignees

Comments

@nus-se-bot
Copy link

I feel that the attendance feature is really incomplete.

image.png

As a teacher, I may need access old records of attendance. This is quite a core feature.
Moreover, they will need to unmark all inorder to use it the next time. What if the teacher has forgotten to mark attendance for that day, how do the teacher know if the attendance are for today or the day before.
Moreover, the export to excel for attendance does not show the date where the attendance is marked too. The date is labeled on XLS file but the attendance taken might not be on that day.

I feel that this is medium as the attendance does not really value add to teachers and requires them to be very merticulous if they want to have to keep records of attendance.

Perhaps it would be nice if attendance is store with date and resets at the end of the day. To retrieve older records, when print perhaps you can print all the old copies and show it in the XLS file.


Labels: type.FeatureFlaw severity.Medium
original: EltonGohJH/ped#10

@fans2619
Copy link

You can go to the link where I wrote some comments and suggestions.

@Wilfredwongkc
Copy link

Include a tip on how to use the mark command with print command

@xyliew25 xyliew25 linked a pull request Nov 1, 2021 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants