Resolve "Create models" #52

Merged
hansegucker merged 7 commits from 1-create-models into master 2023-07-05 19:53:11 +02:00
Owner

Closes #1

Closes #1
Author
Owner

assigned to @nik

assigned to @nik
Author
Owner

added 2 commits

Compare with previous version

added 2 commits <ul><li>1b41e078 - Reformat</li><li>bf621b0b - Add models</li></ul> [Compare with previous version](/AlekSIS/onboarding/AlekSIS-App-Cursus/-/merge_requests/1/diffs?diff_id=27383&start_sha=fd5e00978ae6f6b49bfbbc9df92bc26911f8311c)
Owner

This file should entirely not be there.

This file should entirely not be there.
Author
Owner

added 4 commits

Compare with previous version

added 4 commits <ul><li>5fe8cb0c - Introduce parent subjects</li><li>ebf6cbe9 - Fix related names</li><li>44ad9923 - Depend on newer Core</li><li>34be01b9 - Add initial migration</li></ul> [Compare with previous version](/AlekSIS/onboarding/AlekSIS-App-Cursus/-/merge_requests/1/diffs?diff_id=29057&start_sha=bf621b0b67e48be9d69031c032f683583b77d489)
Author
Owner

resolved all threads

resolved all threads
Author
Owner

marked this merge request as ready

marked this merge request as **ready**
Author
Owner

changed this line in version 3 of the diff

changed this line in [version 3 of the diff](/AlekSIS/onboarding/AlekSIS-App-Cursus/-/merge_requests/1/diffs?diff_id=29059&start_sha=34be01b9c60803f056df01439710cce6c5da41f8#b7af2294cbed3368881b967baaf0f72f7a4e20cb_16_0)
Author
Owner

added 1 commit

Compare with previous version

added 1 commit <ul><li>c20ad193 - Remove legacy files</li></ul> [Compare with previous version](/AlekSIS/onboarding/AlekSIS-App-Cursus/-/merge_requests/1/diffs?diff_id=29059&start_sha=34be01b9c60803f056df01439710cce6c5da41f8)
Author
Owner

mentioned in issue AlekSIS/official/AlekSIS#58

mentioned in issue AlekSIS/official/AlekSIS#58
Member

Why don't you have an on_delete=models.CASCADE here? If you delete a course, its teachers will be deleted as well, which is probably not the intended behaviour.

Why don't you have an `on_delete=models.CASCADE` here? If you delete a course, its teachers will be deleted as well, which is probably _not_ the intended behaviour.
Member

Or maybe i am mistakten...

Or maybe i am mistakten...
Member

yes, i am

yes, i am
Member

resolved all threads

resolved all threads
Owner

mentioned in commit cfe40b085f

mentioned in commit cfe40b085f35b2bbe6d555db56a305ce87f5ac5f
nik merged commit cfe40b085f into master 2023-07-05 19:53:11 +02:00
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-Cursus!52
No description provided.