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

Update selab_script.sql #39

Merged
merged 4 commits into from
Mar 10, 2024
Merged

Update selab_script.sql #39

merged 4 commits into from
Mar 10, 2024

Conversation

DRIESASTER
Copy link
Contributor

team size + check vooraleer toevoegen

DRIESASTER and others added 4 commits March 4, 2024 13:33
* Update selab_script.sql

nr engels, snake_case azureobjectid -> uid

* Update backend/selab_script.sql

Co-authored-by: Xander Bil <[email protected]>

* Update backend/selab_script.sql

Co-authored-by: Xander Bil <[email protected]>

---------

Co-authored-by: Xander Bil <[email protected]>
Co-authored-by: Xander Bil <[email protected]>
Gwn description aan project toevoegen, kan mss wel nice zijn haha
team size + check vooraleer toevoegen
@DRIESASTER DRIESASTER requested a review from xerbalind March 6, 2024 14:13
@@ -46,7 +47,9 @@ CREATE TABLE project (
name TEXT NOT NULL,
subject_id BIGSERIAL NOT NULL,
description TEXT,
FOREIGN KEY (subject_id) REFERENCES subject(id) ON DELETE SET NULL
max_team_size INT NOT NULL DEFAULT 4, -- Added column for maximum team size
Copy link
Contributor Author

Choose a reason for hiding this comment

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

uhmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmm. Oops. Goed dat er reviews zijn.... Ik fix.

@reyniersbram
Copy link
Contributor

kan de PR naar dev ipv naar main :)

@reyniersbram reyniersbram changed the base branch from main to dev March 9, 2024 18:04
@xerbalind xerbalind merged commit 2cd6a7f into dev Mar 10, 2024
1 check passed
@xerbalind xerbalind deleted the db_update branch March 10, 2024 12:10
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.

4 participants