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

Students have more attendances saved than saved by tutor. #161

Closed
Dudrie opened this issue Nov 1, 2019 · 1 comment · Fixed by #1060
Closed

Students have more attendances saved than saved by tutor. #161

Dudrie opened this issue Nov 1, 2019 · 1 comment · Fixed by #1060
Labels
🐞 bug Something isn't working Needs investigation ↗ Priority: High ❓ Still an issue? Maybe an other contribution has also fixed this one.

Comments

@Dudrie
Copy link
Collaborator

Dudrie commented Nov 1, 2019

In some instances (still to be discovered which ones exactly) students have more PRESENT or EXCUSED attendances set in their document / counted by the AttendanceCriteria than the real amout (ie past tutorials).

✔️ Expected behaviour

Counted present/excused states must match the real amount.

⚡ Actual behaviour

Counted present/excused states are higher than the real amount.

🕹️ Steps to reproduce

This still has to be figured out in which situation it happens.

@Dudrie Dudrie added 🐞 bug Something isn't working ℹ Needs more info Further information is needed and/or requested labels Nov 1, 2019
@Dudrie
Copy link
Collaborator Author

Dudrie commented Mar 9, 2020

One has to investigate if this is still an issue for v2.

@Dudrie Dudrie added ❓ Still an issue? Maybe an other contribution has also fixed this one. and removed ℹ Needs more info Further information is needed and/or requested labels Jul 12, 2020
@Morphenoed Morphenoed linked a pull request Nov 16, 2024 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🐞 bug Something isn't working Needs investigation ↗ Priority: High ❓ Still an issue? Maybe an other contribution has also fixed this one.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants