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

Add adding grades to a term and enhance grade logic. #1436

Merged
merged 163 commits into from
Mar 28, 2024
Merged

Conversation

Jonas-Sander
Copy link
Collaborator

@Jonas-Sander Jonas-Sander commented Mar 26, 2024

This PR enhances the grade feature logic and adds a very hacky version of a functioning add grade dialog.

Currently only two grading systems are implemented: 1-6 (+-) and 0-15 points, both have predefined possible values.
In the future I'll need to make also non-discrete grading systems work (e.g. 1-6 with decimals).
In the add grade dialog only 1-6(+-) will work currently though, because the term is always created with this grade system via the UI.

Also adding real courses does not work yet, I'm just using a fake subject.
I wasn't sure what our exact plan was like again (courses with old courses? when does stuff update? how are the subjects connected to a course?) and also I thought its maybe better to do it in a future PR.

grades.webm

nilsreichardt and others added 30 commits March 12, 2024 10:17
Co-authored-by: Jonas Sander <Jonas-Sander@users.noreply.github.com>
@Jonas-Sander Jonas-Sander changed the title Grades logic 2 Add adding grades to a term and enhance grade logic. Mar 27, 2024
@Jonas-Sander Jonas-Sander marked this pull request as ready for review March 27, 2024 17:02
Copy link
Member

@nilsreichardt nilsreichardt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

app/lib/grades/grades_service/src/grading_system.dart Outdated Show resolved Hide resolved
@Jonas-Sander Jonas-Sander added this pull request to the merge queue Mar 28, 2024
Merged via the queue into main with commit 92322f3 Mar 28, 2024
38 checks passed
@Jonas-Sander Jonas-Sander deleted the grades-logic-2 branch March 28, 2024 10:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Changing, updating, adding or removing one or more dependencies. feature: grades Users can add their grades to Sharezone. feature: homework-submissions Submissions can be toggled for homeworks so that pupils can upload their solutions for the teacher. testing
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants