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

Backend route voor het kopiëren van een vak #230

Merged
merged 7 commits into from
Apr 28, 2024

Conversation

Aqua-sc
Copy link
Contributor

@Aqua-sc Aqua-sc commented Apr 26, 2024

What's new 🦜

  • Een route /api/courses/{courseId}/copy
    Deze route kopieert dus een vak (enkel de creator kan dit), meer specifiek is dit wat er mee gekopieert wordt:
  • Het vak zelf
    • CreatedAt wordt aangepast
    • ArchivedAt wordt null
    • Joinkey wordt aangepast
  • Alle groepclusters
  • Alle groepen die niet bij de individuele groepscluster horen (aangezien de gebruikers niet meegekopieert worden)
  • Alle projecten en daarmee:
    • De test van elk project
    • De fysieke bestanden die bij een test horen worden ook gekopieert
  • De user die het vak kopieert (dus de creator) wordt ook als creator toegevoegd aan het gekopieerde vak

TODO om uit draft te kunnen halen

  • ~~wachten tot adding course year to backend #225 gemerged is zodat we het jaar ook mee kunnen kopiëren (eventueel +1 doen)~~~
    -> Ik heb besloten om het jaar niet +1 te doen aangezien dit mss niet altijd gewenst is en je eigenlijk bij "kopiëren" niet verwacht dat er stuff verandert. Eventueel kan hier wel een frontend optie gemaakt worden dat bij kopiëren het vak +1 gaat (en dan gwn na de copy nog een patch doen)

TODO later:

  • Eenmaal de bestanden van de dockertemplates toegevoegd zijn moeten deze ook meegekopieerd worden

@Aqua-sc Aqua-sc linked an issue Apr 26, 2024 that may be closed by this pull request
@Aqua-sc Aqua-sc marked this pull request as ready for review April 27, 2024 06:58
@AWerbrouck AWerbrouck self-requested a review April 27, 2024 15:05
Copy link
Contributor

@AWerbrouck AWerbrouck left a comment

Choose a reason for hiding this comment

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

Paar kleine vragen maar als die beantwoord zijn is het goed.

@Aqua-sc
Copy link
Contributor Author

Aqua-sc commented Apr 28, 2024

De typo is aangepast, als de resolve oké is voor u zo mag je mergen van mij

@Aqua-sc Aqua-sc merged commit 69d4c92 into development Apr 28, 2024
1 check passed
@AWerbrouck AWerbrouck deleted the 229-kopiëren-van-een-vak branch May 23, 2024 17: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.

Kopiëren van een vak
2 participants