EW-1355: Improve CC Import performance.

[mkreuzkam-cap]

Description

Combination of the different changes to move to new way the CC import works.

Links to Tickets or other pull requests

• https://ticketsystem.dbildungscloud.de/browse/EW-1356
• EW-1355: Improve CC Import performance. nuxt-client#4012
• EW-1356: Adjust Cypress tests to new import. e2e-system-tests#469

Approval for review

• DEV: If api was changed - generate-client:server was executed in vue frontend and changes were tested and put in a PR with the same branch name.
• QA: In addition to review, the code has been manually tested (if manual testing is possible)
• All points were discussed with the ticket creator, support-team or product owner. The code upholds all quality guidelines from the PR-template.

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
100.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

[SevenWaysDP]

Hey Jungs, ich habe erstmal zwei Punkte:

1. bitte auf jwt Weitergabe verzichten. Und so umbauen wie in hier: https://github.com/hpi-schul-cloud/schulcloud-server/blob/main/apps/server/src/infra/authorization-client/authorization-client.adapter.ts

2. Warum sind die Client Einzeln implementiert. Bord und Card sind doch unzertrennlich. Warum eigenständige Module? Und sie werden auch nur von Common-Cartrige Module Benutzt. Und Common-Cartridge enthält auch 3 generierte Clients. Man kann doch ein ClientModul zumindest für Board Elemente haben. Bitte noch mal Überdenken.

[alweber-cap]

Hey Jungs, ich habe erstmal zwei Punkte:

1. bitte auf jwt Weitergabe verzichten. Und so umbauen wie in hier: https://github.com/hpi-schul-cloud/schulcloud-server/blob/main/apps/server/src/infra/authorization-client/authorization-client.adapter.ts
2. Warum sind die Client Einzeln implementiert. Bord und Card sind doch unzertrennlich. Warum eigenständige Module? Und sie werden auch nur von Common-Cartrige Module Benutzt. Und Common-Cartridge enthält auch 3 generierte Clients. Man kann doch ein ClientModul zumindest für Board Elemente haben. Bitte noch mal Überdenken.

Hi Sergej,

zu 1: Meinst du den extract Controller: https://github.com/hpi-schul-cloud/schulcloud-server/pull/6013/changes#diff-06537e0ff031837e083b8e27a63e61fb4fd5f49ddb67f86d98c2cebfe9120319R46?
Die Weitergabe in den Clients muss leider so explizit erfolgen, da wir der JWT über die RabbitMQ mitgeben müssen, damit die Aufrufe der Clients auch weiterhin authenticated bleiben. Falls du eine besser Idee hast, wie wir den JWT da durchbekommen, dann gerne her damit :)

zu 2:

• Die Clients im CommonCartridge Modul sind deprecated und werden noch entfernt.
• Ich wollte die Clients gerne seperat halten, einer pro Controller in diesem Fall. Das macht die Clients schlanker und einfacher verständlich meiner Meinung nach. Zudem hat man auch etwas feinere Granularität für Imports in zukünftigen Microservices
• Was wir machen sollten, ist den CardElements Client in ein eigenes Modul zu ziehen, damit es auch konsistent zu der IDee ist.

Wir können gerne morgen oder wann du auch Zeit hast auch nochmal über die Punkte im Detail sprechen