Resolve "[New app] Gradebook" #19

Merged
hansegucker merged 6 commits from 1-new-app-gradebook into main 2024-09-28 13:21:50 +02:00
Member

Closes #1

Closes #1
Author
Member

added 1 commit

  • 1aa62ff7 - Implement models

Compare with previous version

added 1 commit <ul><li>1aa62ff7 - Implement models</li></ul> [Compare with previous version](/AlekSIS/onboarding/AlekSIS-App-Maka/-/merge_requests/1/diffs?diff_id=47771&start_sha=a40762ae44474427134df38992235ed669b78727)
Owner

Probably unique

Probably unique
Owner

PROTECT

PROTECT
Owner

Person missing here

Person missing here
Owner

We are also missing an assignment of grade sets to groups.

We are also missing an assignment of grade sets to groups.
Owner

Option for default rating type, Rating should have grade set

Option for default rating type, Rating should have grade set
Author
Member

added 1 commit

  • d5137e8b - Implement models

Compare with previous version

added 1 commit <ul><li>d5137e8b - Implement models</li></ul> [Compare with previous version](/AlekSIS/onboarding/AlekSIS-App-Maka/-/merge_requests/1/diffs?diff_id=47783&start_sha=1aa62ff7788e44937e978e03cd73ddf6f0bd4ca9)
Owner

Icon field

Icon field
Owner

We are also missing an assignment of grade sets to groups.

We are also missing an assignment of grade sets to groups.
Author
Member

added 1 commit

  • e5e7c420 - Implement models

Compare with previous version

added 1 commit <ul><li>e5e7c420 - Implement models</li></ul> [Compare with previous version](/AlekSIS/onboarding/AlekSIS-App-Maka/-/merge_requests/1/diffs?diff_id=47788&start_sha=d5137e8b822c8da0d9c2297e520a0bd3b8671de2)
Author
Member

resolved all threads

resolved all threads
Owner
    """Single grade choice in a set of grades."""
```suggestion:-0+0 """Single grade choice in a set of grades.""" ```
Owner

I would suggest to different unique constraint (grade_set, name and grade_set, value).

I would suggest to different unique constraint (grade_set, name and grade_set, value).
Owner

We should use 1 as default.

We should use 1 as default.
Owner

We are also missing an assignment of grade sets to groups.

We are also missing an assignment of grade sets to groups.
Author
Member

changed this line in version 4 of the diff

changed this line in [version 4 of the diff](/AlekSIS/onboarding/AlekSIS-App-Maka/-/merge_requests/1/diffs?diff_id=47828&start_sha=e5e7c420af9bf13a79ee3cc8a000cd1a65a5a416#39b3c490449f2df4c1f240fd20106a66f87f8a77_24_24)
Author
Member

changed this line in version 4 of the diff

changed this line in [version 4 of the diff](/AlekSIS/onboarding/AlekSIS-App-Maka/-/merge_requests/1/diffs?diff_id=47828&start_sha=e5e7c420af9bf13a79ee3cc8a000cd1a65a5a416#39b3c490449f2df4c1f240fd20106a66f87f8a77_40_40)
Author
Member

changed this line in version 4 of the diff

changed this line in [version 4 of the diff](/AlekSIS/onboarding/AlekSIS-App-Maka/-/merge_requests/1/diffs?diff_id=47828&start_sha=e5e7c420af9bf13a79ee3cc8a000cd1a65a5a416#39b3c490449f2df4c1f240fd20106a66f87f8a77_71_75)
Author
Member

added 3 commits

  • fc85957d - Implement models
  • 52c8bc8d - Adjust pyproject and tox configuration
  • 5cedd83c - Implement CRUD schema

Compare with previous version

added 3 commits <ul><li>fc85957d - Implement models</li><li>52c8bc8d - Adjust pyproject and tox configuration</li><li>5cedd83c - Implement CRUD schema</li></ul> [Compare with previous version](/AlekSIS/onboarding/AlekSIS-App-Maka/-/merge_requests/1/diffs?diff_id=47828&start_sha=e5e7c420af9bf13a79ee3cc8a000cd1a65a5a416)
Author
Member

resolved all threads

resolved all threads
Author
Member

added 1 commit

  • ab06b642 - Implement CRUD schema

Compare with previous version

added 1 commit <ul><li>ab06b642 - Implement CRUD schema</li></ul> [Compare with previous version](/AlekSIS/onboarding/AlekSIS-App-Maka/-/merge_requests/1/diffs?diff_id=47848&start_sha=5cedd83cbb3a4ca49b01a376e35884e82d36f24b)
Owner

Unique constraint with condition on True

Unique constraint with condition on `True`
Owner

EffortType

`EffortType`
Owner
class Effort(ExtensibleModel):
```suggestion:-0+0 class Effort(ExtensibleModel): ```
Author
Member

changed this line in version 6 of the diff

changed this line in [version 6 of the diff](/AlekSIS/onboarding/AlekSIS-App-Maka/-/merge_requests/1/diffs?diff_id=47870&start_sha=ab06b642d598a64d15855bdcbb77fbf6118d0ea2#39b3c490449f2df4c1f240fd20106a66f87f8a77_72_72)
Author
Member

added 3 commits

Compare with previous version

added 3 commits <ul><li>bbfa7802 - Implement models</li><li>f2f527b0 - Adjust pyproject and tox configuration</li><li>f75d5984 - Implement CRUD schema</li></ul> [Compare with previous version](/AlekSIS/onboarding/AlekSIS-App-Maka/-/merge_requests/1/diffs?diff_id=47870&start_sha=ab06b642d598a64d15855bdcbb77fbf6118d0ea2)
Member

added 2 commits

  • 67f7f7c1 - Fix typos
  • 324b67a0 - Rename imported Models to avoid confusion with graphql types

Compare with previous version

added 2 commits <ul><li>67f7f7c1 - Fix typos</li><li>324b67a0 - Rename imported Models to avoid confusion with graphql types</li></ul> [Compare with previous version](/AlekSIS/onboarding/AlekSIS-App-Maka/-/merge_requests/1/diffs?diff_id=47872&start_sha=f75d59840fa95b38bcf845f7f3b3d1b28970873a)
Author
Member

added 1 commit

  • 752416ea - Add unique constraint for EffortType default

Compare with previous version

added 1 commit <ul><li>752416ea - Add unique constraint for EffortType default</li></ul> [Compare with previous version](/AlekSIS/onboarding/AlekSIS-App-Maka/-/merge_requests/1/diffs?diff_id=47905&start_sha=324b67a06ab90743c2dade917770791682d03bb0)
Owner

resolved all threads

resolved all threads
Owner

marked this merge request as ready

marked this merge request as **ready**
Owner

mentioned in commit a0e88ad2ef

mentioned in commit a0e88ad2ef67757973dbecfdc66dc4e12d46e9e3
Sign in to join this conversation.
No reviewers
No milestone
No project
No assignees
3 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set

Reference
aleksis/AlekSIS-App-Maka!19
No description provided.