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

Model update #55

Merged
merged 14 commits into from
Mar 7, 2024
Merged

Model update #55

merged 14 commits into from
Mar 7, 2024

Conversation

BramMeir
Copy link
Contributor

@BramMeir BramMeir commented Mar 6, 2024

closes #49

Fix that project has a link to its groups.

Added some extra fields:

  • Max score that can be achieved on a project
  • Max number of members that a group can have

Refactor of checks model: this now consists of two parts: structure checks and extra checks. The structure checks are for the folder structure and obligated/forbidden extensions. The extra checks are for automated tests in a script form.

Refactor of submission model: status field for the obligated structure checks, and a link to the result of the extra checks that are succeeded/failed.

Updated ER: https://lucid.app/lucidchart/19d9d513-7c7d-4255-9495-ea11be8080ca/edit?invitationId=inv_581f5c04-9398-4d02-be58-4c009050f5be&page=0_0#

@BramMeir BramMeir added this to the Backend milestone milestone Mar 6, 2024
@BramMeir BramMeir self-assigned this Mar 6, 2024
backend/api/urls.py Outdated Show resolved Hide resolved
Copy link
Contributor

@tyboro2002 tyboro2002 left a comment

Choose a reason for hiding this comment

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

looks good

backend/api/urls.py Outdated Show resolved Hide resolved
Copy link
Contributor

@EwoutV EwoutV left a comment

Choose a reason for hiding this comment

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

👌

@EwoutV EwoutV merged commit 31672e5 into development Mar 7, 2024
2 checks passed
@EwoutV EwoutV deleted the model_update branch March 7, 2024 09:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

backend(api): Update database models
3 participants