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

Studentennummer + tijdstip waarop project zichtbaar wordt #266

Merged
merged 8 commits into from
May 18, 2024

Conversation

Aqua-sc
Copy link
Contributor

@Aqua-sc Aqua-sc commented May 17, 2024

Studentennummer

  • Een veld studentNumber is toegevoegd aan:
    • UserReferenceJson: wordt teruggegeven bv. wanneer alle users v/e course/group opgelijst worden
    • UserJson: wordt teruggegeven indien de user wordt opgevraagd
  • Dit veld is null indien de user die het opvraagt niet een courseadmin is. Op het moment dat bij een vak de teacher + assistents worden opgelijst is het ook default null
  • ❗Momenteel wordt het studentennummer nog niet correct ingevuld aangezien dit met de huidige authenticatie nog niet beschikbaar is. Eenmaal de express server gekoppelt is zou het meteen moeten werken

Project zichtbaar tijdstip

  • Een optioneel veld visibleAfter (TIMESTAMP) is toegevoegd aan de POST/PATCH/PUT v/e project:
    • POST/PUT: indien het veld niet meegegeven wordt het op null gezet
    • PATCH: indien het veld niet meegegeven wordt blijft het wat het was
    • Bij PUT/PATCH: indien visibleAfter al gepasseert is maar isVisible toch false zou zijn wordt dit aangepast naar true
  • Een veld visibleAfter is toegevoegd aan de ProjectResponseJson, kan perfect null zijn of kan een timestamp bevatten
  • Indien een project opgevraagt wordt (individueel of via ../projects) wordt indien nodig op basis van visibleAfter, isVisible geupdate

Lock groups tijdstip

  • Analoog aan hierboven is een optioneel veld toegevoegd lockGroupsAfter aan zowel de request als reponse json van groepClusters
  • Indien dit tijdstip gepasseerd is en een user een groep probeert te joinen/leaven zal dit een 403 teruggeven. Een course admin can wel nog steeds users toevoegen/verwijderen aan/uit groepen

@Aqua-sc Aqua-sc requested a review from usserwoutV2 May 17, 2024 18:27
@Aqua-sc Aqua-sc marked this pull request as ready for review May 17, 2024 19:19
@usserwoutV2
Copy link
Contributor

Voor frontend ziet het er goed uit.

@Aqua-sc Aqua-sc marked this pull request as draft May 18, 2024 04:15
@Aqua-sc
Copy link
Contributor Author

Aqua-sc commented May 18, 2024

Even in draft gezet aangezien ik nog een gelijkaardig veldje lockGroupsAfter ga toevoegen om te voorkomen dat users andere groepen kunnen joinen en hun submissions zien

@usserwoutV2 usserwoutV2 marked this pull request as ready for review May 18, 2024 08:37
@usserwoutV2 usserwoutV2 merged commit e8e2d3b into development May 18, 2024
1 check passed
@usserwoutV2 usserwoutV2 deleted the feature/studentnummer_en_projectvisibletime branch May 18, 2024 08:37
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.

2 participants