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

Meerdere gebruikers aan groepen toevoegen in 1 keer #231

Closed
usserwoutV2 opened this issue Apr 28, 2024 · 1 comment · Fixed by #252
Closed

Meerdere gebruikers aan groepen toevoegen in 1 keer #231

usserwoutV2 opened this issue Apr 28, 2024 · 1 comment · Fixed by #252
Assignees
Labels

Comments

@usserwoutV2
Copy link
Contributor

In frontend heb ik nu iets gemaakt dat een course admin de groepen zelf kan verdelen. Om te vermijden dat ik een groot aantal POST requests moet uitvoeren zou het handig zijn om 1 PUT request te maken om alle groepen van een cluster te overschrijven. Ik bewaar de state van alle groepen in de frontend als volgt:

type GroupCluster = {
  [groupId:string]: number[] // userIds
}

Dus bv:

{
 "1": [1,2,3,4],
 "2": [],
 "3": [5]
}

Als het gemakkelijker is om dit op een andere manier door te geven aan de backend, dan kan ik dit nog veranderen.

Scherm­afbeelding 2024-04-28 om 14 24 41
@arnedierick arnedierick self-assigned this May 8, 2024
@arnedierick
Copy link
Contributor

Ik denk dat dit inderdaad een nuttige issue is die volgens mij niet zo moeilijk is om te implementeren in de backend. Ik begin eraan

@arnedierick arnedierick linked a pull request May 9, 2024 that will close this issue
@Aqua-sc Aqua-sc closed this as completed May 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants