Not storing groups for documentations is kind of a problem #351

New issue

Closed

opened 2025-01-05 20:01:52 +01:00 by hansegucker · 6 comments

hansegucker commented

2025-01-05 20:01:52 +01:00

Owner

Today, I stumbled over a new edge case:

There was a lesson event without a course, but with a group (groups field) which has been documented in the coursebook (leads to documentation without any course or group information).
There is a student who has been marked as absent in this documentation.
Now, in the statistics overview of the group, the absent participation status of the student isn't counted because the documentation has no course.
In the personal overview of the student, the absence is counted.

Conclusion: It's a problem to only store the course with the documentation.

Today, I stumbled over a new edge case: 1. There was a lesson event without a course, but with a group (groups field) which has been documented in the coursebook (leads to documentation without any course or group information). 2. There is a student who has been marked as absent in this documentation. 3. Now, in the statistics overview of the group, the absent participation status of the student isn't counted because the documentation has no course. 4. In the personal overview of the student, the absence is counted. Conclusion: It's a problem to only store the course with the documentation.

hansegucker commented

2025-01-08 20:47:33 +01:00

Author

Owner

Dev meeting decided: We store groups with the documentation as source of truth. Needs a migration to set groups for existings documentations (set from course, if course isn't set from amended lesson event).

Dev meeting decided: We store `groups` with the documentation as source of truth. Needs a migration to set groups for existings documentations (set from course, if course isn't set from amended lesson event).

hansegucker commented

2025-01-13 16:40:41 +01:00

Author

Owner

assigned to @permcu

hansegucker commented

2025-01-22 20:47:14 +01:00

Author

Owner

assigned to @hansegucker and unassigned @permcu

hansegucker commented

2025-01-24 11:59:42 +01:00

Author

Owner

created branch 351-not-storing-groups-for-documentations-is-kind-of-a-problem to address this issue

created branch [`351-not-storing-groups-for-documentations-is-kind-of-a-problem`](/AlekSIS/official/AlekSIS-App-Alsijil/-/compare/master...351-not-storing-groups-for-documentations-is-kind-of-a-problem) to address this issue

hansegucker commented

2025-01-24 12:00:08 +01:00

Author

Owner

mentioned in merge request !866