Skip to content

[Frontend] - Not all students need to be assigned to an assignment#633

Merged
thomasdejaeghere merged 7 commits intodevelopmentfrom
616-divide-group-fix
May 20, 2025
Merged

[Frontend] - Not all students need to be assigned to an assignment#633
thomasdejaeghere merged 7 commits intodevelopmentfrom
616-divide-group-fix

Conversation

@brentjan
Copy link
Contributor

@brentjan brentjan commented May 20, 2025

Fixed such that there is no need for all students of a class to be divided into groups for an assignment, the members left in the members list aren't assigned to the assignment.

edit
fixed restrictions on assignment creation (you need students) and allowed >0 members in the assignment

create.flow.mp4

@brentjan brentjan added this to the Milestone 3 milestone May 20, 2025
@brentjan brentjan self-assigned this May 20, 2025
@brentjan brentjan added enhancement New feature or request frontend labels May 20, 2025
// width: inherit;
// padding-left: 10%;
// }

Copy link
Contributor

Choose a reason for hiding this comment

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

@lennertdr does this break the responsivity that you put up last night?

Copy link
Contributor

Choose a reason for hiding this comment

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

Nope its alright, responsive part lies in the media query

@lennertdr lennertdr self-assigned this May 20, 2025

accepted$.subscribe((response) => {
if(response) this.openSnackBar(this.acceptedMessage);
if (response) this.openSnackBar(this.acceptedMessage);
Copy link
Contributor

Choose a reason for hiding this comment

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

i love my linter

Copy link
Contributor

Choose a reason for hiding this comment

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

Linter my beloved

@thomasdejaeghere thomasdejaeghere merged commit 401e70d into development May 20, 2025
2 checks passed
@thomasdejaeghere thomasdejaeghere deleted the 616-divide-group-fix branch May 20, 2025 14:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request frontend

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants