Peer reviewing functionality

To be able to better differentiate high quality exercises it was proposed to implement a peer reviewing functionality and once an exercise is peer reviewed it should display that as a badge or something similar

added badges discussion feature labels

assigned to @root

assigned to @eduard.frankford and unassigned @root

changed milestone to %2023 Feb Sprint

mentioned in issue #210 (closed)

mentioned in merge request !164 (merged)

changed milestone to %2023 März Sprint

Bin noch nicht bis zur Peer-Review-Funktionalität vorgedrungen :-(

Schon Erste Rückmeldungen:

0. Ich habe mir erlaubt, nochmals den aktuellen development-Zweig reinzumergen. (Im vorherigen Development-Stand waren ein paar peinliche Fehler drin ;-) )
1. Der Port in webpack.custom.js war auf 9001 eingestellt. Funktioniert mit dem Entwicklungs-KeyCloak nicht.
2. Auch in application-dev.yml waren ein paar Sachen auf Produktion eingestellt?
3. Es scheinen verschiedene Änderungen z.B. in UserResource vorgenommen worden zu sein, die alte Funktionalität (User Management) kaputt gemacht haben. Außerdem wurden wohl Security Constaints gelockert. Bist Du sicher, dass wir damit die Sicherheit nicht kompromittieren?

1. Der Port in webpack.custom.js war auf 9001 eingestellt. Funktioniert mit dem Entwicklungs-KeyCloak nicht.

Geht jetzt :-)

2. Auch in application-dev.yml waren ein paar Sachen auf Produktion eingestellt?

Was denn genau? Keycloak habe ich wieder auf dev gestellt

3. Es scheinen verschiedene Änderungen z.B. in UserResource vorgenommen worden zu sein, die alte Funktionalität (User Management) kaputt gemacht haben. Außerdem wurden wohl Security Constaints gelockert. Bist Du sicher, dass wir damit die Sicherheit nicht kompromittieren?

Hab ich wieder repariert

Jetzt schon mal erste Rückmeldungen zum Peer Reviewing:

1. Im Nutzermanagement steht:

   

   Also sollte ich wohl automatisch (?) aktiviert sein.

   Im Peer Reviewing ist der Button disabled?

   

2. Der Titel scheint nicht übersetzt:

   

3. In Review Admin Oberfläche: Fehlermeldung "... is already in the database ...". Kann aber eigenlich nicht sein

   

4. Kleinigkeit: Es kann nur eine Aufgabe ausgewählt werden: Besser Radiobutttons statt Checkboxes?

   

5. Mailversand: Momentan wird die Absenderadresse aus der Umgebungsvariable "MAIL_FROM" gezogen. Würde ich eher in die application.yml legen

More to come

1. sollte jetzt gehen
2. Hab den Titel richtig gesetzt
3. Hab ich auch behoben
4. Wird jetzt als radiobutton angezeigt
5. Hab ich auch umgesetzt

mentioned in issue #375 (closed)

changed milestone to %2023 April Sprint

done

closed