DMC/scipro
7
0
Fork 0
Code Issues31 Pull Requests4 Actions Projects1 Releases Wiki Activity

Allow supervisors to request improvements from final seminar opponents #78

Merged
niat8586 merged 41 commits from opponent-completion into develop 2025-03-05 10:05:38 +01:00
Conversation 8 Commits 41 Files Changed 53 +1153 -263
ansv7779 commented 2025-01-13 16:28:56 +01:00
Owner

Fixes

How to test

  1. (Optional) Log in as sid@example.com and submit an opposition report
    1. Go to the tab "Opposition & Active participation"
    2. Open the opposition "Putting the it in supervising" on the right
    3. Submit the report
  2. Log in as eric@example.com
  3. Go to the final seminar in the "Putting the it in supervising" project (or follow the notification if you did step 1)
  4. Request improvements
  5. Log in as sid@example.com
  6. Follow the notification to submit the new opposition report

Click the "Re-run all jobs" button (top right) on https://gitea.dsv.su.se/DMC/scipro/actions/runs/457 to reset the database. It takes a few minutes.

Fixes #36 ## How to test 1. (Optional) Log in as `sid@example.com` and submit an opposition report 1. Go to the tab "Opposition & Active participation" 2. Open the opposition "Putting the it in supervising" on the right 3. Submit the report 2. Log in as `eric@example.com` 3. Go to the final seminar in the "Putting the it in supervising" project (or follow the notification if you did step 1) 4. Request improvements 5. Log in as `sid@example.com` 6. Follow the notification to submit the new opposition report Click the "Re-run all jobs" button (top right) on https://gitea.dsv.su.se/DMC/scipro/actions/runs/457 to reset the database. It takes a few minutes.
ansv7779 added 2 commits 2025-01-13 16:28:57 +01:00
gitea-actions commented 2025-01-13 16:32:42 +01:00
First-time contributor

Deployed to https://scipro-opponent-completion.branch.dsv.su.se

Deployed to https://scipro-opponent-completion.branch.dsv.su.se
ansv7779 added 3 commits 2025-01-14 15:33:23 +01:00 
Changed how data is moved from the opposition to the grading report

Fixed a completely broken integration test
ansv7779 added 3 commits 2025-01-17 12:36:22 +01:00
ansv7779 added 2 commits 2025-01-17 13:19:43 +01:00
ansv7779 added 1 commit 2025-01-17 15:18:41 +01:00
ansv7779 added 1 commit 2025-01-17 15:21:21 +01:00
ansv7779 force-pushed opponent-completion from 7a3b8d4a98 to 50a082adb5 2025-01-20 10:50:34 +01:00 Compare
ansv7779 force-pushed opponent-completion from 50a082adb5 to 7bb37407d6 2025-01-20 14:22:09 +01:00 Compare
ansv7779 added 1 commit 2025-01-20 14:24:30 +01:00
ansv7779 added 1 commit 2025-01-21 14:55:06 +01:00
ansv7779 added 1 commit 2025-01-23 17:47:00 +01:00
ansv7779 added 1 commit 2025-01-23 21:17:44 +01:00
ansv7779 added 1 commit 2025-01-24 15:16:08 +01:00 
The entire system is in english so have two paragraphs in swedish made no sense.
ansv7779 added 1 commit 2025-01-24 15:47:15 +01:00
ansv7779 added 2 commits 2025-02-06 16:38:24 +01:00
ansv7779 added 3 commits 2025-02-07 12:40:34 +01:00
ansv7779 added 1 commit 2025-02-07 15:28:16 +01:00
ansv7779 added 2 commits 2025-02-10 10:48:59 +01:00 
The reviewer report has long since been removed and the supervisors report differs too much and had been decoupled already.
ansv7779 changed title from WIP: Allow supervisors to request improvements from final seminar opponents to Allow supervisors to request improvements from final seminar opponents 2025-02-10 10:52:53 +01:00
ansv7779 added the
po
needed
 label 2025-02-10 10:53:27 +01:00
ansv7779 added 1 commit 2025-02-10 14:35:42 +01:00
niat8586 requested review from niat8586 2025-02-11 08:53:35 +01:00
niat8586 commented 2025-02-11 10:07:48 +01:00
Owner

After a request for improvements

Logged in as sid@example.com after the request for improvements has been done.

Shouldn't the view in the Opponents reflect that a request for improvements has been made? Look at the below screen shot.
image.png

Now it just says that no opposition report has been submitted yet. Could this be confusing to the opponent?

After a request for improvements Logged in as sid@example.com after the request for improvements has been done. Shouldn't the view in the Opponents reflect that a request for improvements has been made? Look at the below screen shot. ![image.png](/attachments/dc737f5c-c7aa-44f4-84b7-ae602fb3bef4) Now it just says that no opposition report has been submitted yet. Could this be confusing to the opponent?
image.png
16 KiB
ansv7779 added 2 commits 2025-02-11 15:37:48 +01:00 
Now takes the opponent directly to the form where they can fill in the report and view the feedback from the seminar supervisor.
ansv7779 added 1 commit 2025-02-11 15:47:26 +01:00
ansv7779 commented 2025-02-11 15:48:39 +01:00
Author
Owner

Shouldn't the view in the Opponents reflect that a request for improvements has been made? Look at the below screen shot.
image.png

Now it just says that no opposition report has been submitted yet. Could this be confusing to the opponent?

I've added an alert to notify the opponent that improvements have been requested. I also made the notification take you directly to the form to make the UX for the opponent even better.

> Shouldn't the view in the Opponents reflect that a request for improvements has been made? Look at the below screen shot. > ![image.png](/attachments/dc737f5c-c7aa-44f4-84b7-ae602fb3bef4) > > Now it just says that no opposition report has been submitted yet. Could this be confusing to the opponent? > I've added an alert to notify the opponent that improvements have been requested. I also made the notification take you directly to the form to make the UX for the opponent even better.
ansv7779 added 1 commit 2025-02-12 11:24:32 +01:00 
# Conflicts:
#	core/src/main/java/se/su/dsv/scipro/DataInitializer.java
ansv7779 added 1 commit 2025-02-12 13:45:54 +01:00
niat8586 approved these changes 2025-02-12 15:10:23 +01:00
Dismissed
niat8586 left a comment
Owner

👍

Really like the changes to the ui.
Notification takes you directly to the request to improve the opposition.
Also when going to the opposition the added information that the supervisor has requested improvements makes it more clear now that you need to do.

Good work

👍 Really like the changes to the ui. Notification takes you directly to the request to improve the opposition. Also when going to the opposition the added information that the supervisor has requested improvements makes it more clear now that you need to do. Good work
andan commented 2025-02-14 08:01:50 +01:00
Owner

PO Review

  1. Inloggad som seminarieordförande, när man skall ge poäng för oppositionsrapport. Texten om krav för resp. poäng (Requirement for 1 point: that the opposition report provides a short summary of the evaluated thesis, that it deliberates about the scientific basis, etc) verkar inte vara samma text som i de nya betygskriterier. Den behöver uppdateras. Troligen en gammal hårdkodad text?
    https://nextilearn.dsv.su.se/pluginfile.php/25609/mod_resource/content/3/Thesis_Instructions_2025.pdf

  2. Inloggad som seminarieordförande, när man skall ge poäng för oppositionsrapport. Om man ger noll poäng och trycker på ”submit”: detta innebär att rapporten blir underkänd och studenten behöver anmäla sig till ett nytt slutsem, samt skriva en helt ny oppositionsrapport etc (om jag inte minns fel). Med tanke på effekten, borde vi kanske varna med ett meddelande. När man väljer noll poäng och klickar på submit: ”0 points implies failing the student’s opposition report. The student will have to sign up for a new seminar and write a new opposition report. Are you sure you want to continue?” Eller något liknande 😊

  3. Inloggad som seminarieordförande, när man skall ge poäng för oppositionsrapport. När man tryckt ”Request Improvements” och skickat feedback till studenten. Som handledare/seminarieordförande kan man fortfarande välja att ge poäng. Vad händer då? Man borde frysa den möjligheten tills en ny uppdaterad version skickas av studenten.

  4. När handledaren begärt komplettering och studenten skickat in kompletteringen. När handledaren ger sen godkända poäng (1-2), skickas ingen (ny) notifikation till studenten? Inloggad som student kunde jag bara se notifikationen för avslag.

  5. Inloggad som seminarieordförande, när man skall ge poäng för oppositionsrapport. Om studenten skickar en bilaga som oppositionsrapport, syns enbart datumet för uppladdningen. Kan vi lägga till även tidpunkten för uppladdningen? Som vi gjort med final thesis-uppladdningar.

**PO Review** 1) **Inloggad som seminarieordförande, när man skall ge poäng för oppositionsrapport**. Texten om krav för resp. poäng (_Requirement for 1 point: that the opposition report provides a short summary of the evaluated thesis, that it deliberates about the scientific basis, etc_) verkar inte vara samma text som i de nya betygskriterier. Den behöver uppdateras. Troligen en gammal hårdkodad text? https://nextilearn.dsv.su.se/pluginfile.php/25609/mod_resource/content/3/Thesis_Instructions_2025.pdf 2) **Inloggad som seminarieordförande, när man skall ge poäng för oppositionsrapport**. Om man ger noll poäng och trycker på ”submit”: detta innebär att rapporten blir underkänd och studenten behöver anmäla sig till ett nytt slutsem, samt skriva en helt ny oppositionsrapport etc (om jag inte minns fel). Med tanke på effekten, borde vi kanske varna med ett meddelande. När man väljer noll poäng och klickar på submit: ”_0 points implies failing the student’s opposition report. The student will have to sign up for a new seminar and write a new opposition report. Are you sure you want to continue?_” Eller något liknande 😊 3) **Inloggad som seminarieordförande, när man skall ge poäng för oppositionsrapport.** När man tryckt ”Request Improvements” och skickat feedback till studenten. Som handledare/seminarieordförande kan man fortfarande välja att ge poäng. **Vad händer då?** Man borde frysa den möjligheten tills en ny uppdaterad version skickas av studenten. 4) **När handledaren begärt komplettering och studenten skickat in kompletteringen.** När handledaren ger sen godkända poäng (1-2), skickas ingen (ny) notifikation till studenten? Inloggad som student kunde jag bara se notifikationen för avslag. 5) **Inloggad som seminarieordförande, när man skall ge poäng för oppositionsrapport**. Om studenten skickar en bilaga som oppositionsrapport, syns enbart datumet för uppladdningen. Kan vi lägga till även tidpunkten för uppladdningen? Som vi gjort med final thesis-uppladdningar.
ansv7779 added
po
changes
 and removed
po
needed
 labels 2025-02-18 10:00:18 +01:00
ansv7779 added 1 commit 2025-02-18 10:04:41 +01:00 
# Conflicts:
#	core/src/main/java/se/su/dsv/scipro/DataInitializer.java
ansv7779 added 6 commits 2025-02-18 15:19:29 +01:00 
It interfered with grading report templates due to having the grading module activated but no template defined. Could have removed the module but it serves no purpose since all PhD-related functionality was removed a long time ago.
ansv7779 commented 2025-02-18 15:45:12 +01:00
Author
Owner
  1. Inloggad som seminarieordförande, när man skall ge poäng för oppositionsrapport. Texten om krav för resp. poäng (Requirement for 1 point: that the opposition report provides a short summary of the evaluated thesis, that it deliberates about the scientific basis, etc) verkar inte vara samma text som i de nya betygskriterier. Den behöver uppdateras. Troligen en gammal hårdkodad text?
    https://nextilearn.dsv.su.se/pluginfile.php/25609/mod_resource/content/3/Thesis_Instructions_2025.pdf

Texten tas från den mall som finns på våra test-servrar, se https://scipro-opponent-completion.branch.dsv.su.se/admin/project/grading/template/view?id=1. Ingen text är hårdkodad.

  1. Inloggad som seminarieordförande, när man skall ge poäng för oppositionsrapport. Om man ger noll poäng och trycker på ”submit”: detta innebär att rapporten blir underkänd och studenten behöver anmäla sig till ett nytt slutsem, samt skriva en helt ny oppositionsrapport etc (om jag inte minns fel). Med tanke på effekten, borde vi kanske varna med ett meddelande. När man väljer noll poäng och klickar på submit: ”0 points implies failing the student’s opposition report. The student will have to sign up for a new seminar and write a new opposition report. Are you sure you want to continue?” Eller något liknande 😊

Känns lite out-of-scope. Lägg ett nytt ärende tycker jag.

  1. Inloggad som seminarieordförande, när man skall ge poäng för oppositionsrapport. När man tryckt ”Request Improvements” och skickat feedback till studenten. Som handledare/seminarieordförande kan man fortfarande välja att ge poäng. Vad händer då? Man borde frysa den möjligheten tills en ny uppdaterad version skickas av studenten.

Fixat.

  1. När handledaren begärt komplettering och studenten skickat in kompletteringen. När handledaren ger sen godkända poäng (1-2), skickas ingen (ny) notifikation till studenten? Inloggad som student kunde jag bara se notifikationen för avslag.

Fixat.

  1. Inloggad som seminarieordförande, när man skall ge poäng för oppositionsrapport. Om studenten skickar en bilaga som oppositionsrapport, syns enbart datumet för uppladdningen. Kan vi lägga till även tidpunkten för uppladdningen? Som vi gjort med final thesis-uppladdningar.

Fixat.

> 1) **Inloggad som seminarieordförande, när man skall ge poäng för oppositionsrapport**. Texten om krav för resp. poäng (_Requirement for 1 point: that the opposition report provides a short summary of the evaluated thesis, that it deliberates about the scientific basis, etc_) verkar inte vara samma text som i de nya betygskriterier. Den behöver uppdateras. Troligen en gammal hårdkodad text? > https://nextilearn.dsv.su.se/pluginfile.php/25609/mod_resource/content/3/Thesis_Instructions_2025.pdf Texten tas från den mall som finns på våra test-servrar, se https://scipro-opponent-completion.branch.dsv.su.se/admin/project/grading/template/view?id=1. Ingen text är hårdkodad. > 2) **Inloggad som seminarieordförande, när man skall ge poäng för oppositionsrapport**. Om man ger noll poäng och trycker på ”submit”: detta innebär att rapporten blir underkänd och studenten behöver anmäla sig till ett nytt slutsem, samt skriva en helt ny oppositionsrapport etc (om jag inte minns fel). Med tanke på effekten, borde vi kanske varna med ett meddelande. När man väljer noll poäng och klickar på submit: ”_0 points implies failing the student’s opposition report. The student will have to sign up for a new seminar and write a new opposition report. Are you sure you want to continue?_” Eller något liknande 😊 Känns lite out-of-scope. Lägg ett nytt ärende tycker jag. > 3) **Inloggad som seminarieordförande, när man skall ge poäng för oppositionsrapport.** När man tryckt ”Request Improvements” och skickat feedback till studenten. Som handledare/seminarieordförande kan man fortfarande välja att ge poäng. **Vad händer då?** Man borde frysa den möjligheten tills en ny uppdaterad version skickas av studenten. Fixat. > 4) **När handledaren begärt komplettering och studenten skickat in kompletteringen.** När handledaren ger sen godkända poäng (1-2), skickas ingen (ny) notifikation till studenten? Inloggad som student kunde jag bara se notifikationen för avslag. Fixat. > 5) **Inloggad som seminarieordförande, när man skall ge poäng för oppositionsrapport**. Om studenten skickar en bilaga som oppositionsrapport, syns enbart datumet för uppladdningen. Kan vi lägga till även tidpunkten för uppladdningen? Som vi gjort med final thesis-uppladdningar. Fixat.
ansv7779 added
po
needed
 and removed
po
changes
 labels 2025-02-18 15:45:24 +01:00
ansv7779 added 1 commit 2025-02-21 14:17:39 +01:00 
# Conflicts:
#	core/src/main/java/se/su/dsv/scipro/DataInitializer.java
ansv7779 added 1 commit 2025-03-04 12:44:30 +01:00 
# Conflicts:
#	test-data/src/main/java/se/su/dsv/scipro/testdata/DataInitializer.java
andan commented 2025-03-05 08:26:40 +01:00
Owner
  1. Inloggad som seminarieordförande, när man skall ge poäng för oppositionsrapport. Texten om krav för resp. poäng (Requirement for 1 point: that the opposition report provides a short summary of the evaluated thesis, that it deliberates about the scientific basis, etc) verkar inte vara samma text som i de nya betygskriterier. Den behöver uppdateras. Troligen en gammal hårdkodad text?
    https://nextilearn.dsv.su.se/pluginfile.php/25609/mod_resource/content/3/Thesis_Instructions_2025.pdf

Texten tas från den mall som finns på våra test-servrar, se https://scipro-opponent-completion.branch.dsv.su.se/admin/project/grading/template/view?id=1. Ingen text är hårdkodad.

Bra, det verkar vara fixat nu!

  1. Inloggad som seminarieordförande, när man skall ge poäng för oppositionsrapport. Om man ger noll poäng och trycker på ”submit”: detta innebär att rapporten blir underkänd och studenten behöver anmäla sig till ett nytt slutsem, samt skriva en helt ny oppositionsrapport etc (om jag inte minns fel). Med tanke på effekten, borde vi kanske varna med ett meddelande. När man väljer noll poäng och klickar på submit: ”0 points implies failing the student’s opposition report. The student will have to sign up for a new seminar and write a new opposition report. Are you sure you want to continue?” Eller något liknande 😊

Känns lite out-of-scope. Lägg ett nytt ärende tycker jag.

OK jag skriver ett nytt ärende.

  1. Inloggad som seminarieordförande, när man skall ge poäng för oppositionsrapport. När man tryckt ”Request Improvements” och skickat feedback till studenten. Som handledare/seminarieordförande kan man fortfarande välja att ge poäng. Vad händer då? Man borde frysa den möjligheten tills en ny uppdaterad version skickas av studenten.

Fixat.

Funkar bra! Tack!

  1. När handledaren begärt komplettering och studenten skickat in kompletteringen. När handledaren ger sen godkända poäng (1-2), skickas ingen (ny) notifikation till studenten? Inloggad som student kunde jag bara se notifikationen för avslag.

Fixat.

Funkar bra! Tack!

  1. Inloggad som seminarieordförande, när man skall ge poäng för oppositionsrapport. Om studenten skickar en bilaga som oppositionsrapport, syns enbart datumet för uppladdningen. Kan vi lägga till även tidpunkten för uppladdningen? Som vi gjort med final thesis-uppladdningar.

Fixat.

Funkar bra! Tack!

> > 1) **Inloggad som seminarieordförande, när man skall ge poäng för oppositionsrapport**. Texten om krav för resp. poäng (_Requirement for 1 point: that the opposition report provides a short summary of the evaluated thesis, that it deliberates about the scientific basis, etc_) verkar inte vara samma text som i de nya betygskriterier. Den behöver uppdateras. Troligen en gammal hårdkodad text? > > https://nextilearn.dsv.su.se/pluginfile.php/25609/mod_resource/content/3/Thesis_Instructions_2025.pdf > > Texten tas från den mall som finns på våra test-servrar, se https://scipro-opponent-completion.branch.dsv.su.se/admin/project/grading/template/view?id=1. Ingen text är hårdkodad. > Bra, det verkar vara fixat nu! > > 2) **Inloggad som seminarieordförande, när man skall ge poäng för oppositionsrapport**. Om man ger noll poäng och trycker på ”submit”: detta innebär att rapporten blir underkänd och studenten behöver anmäla sig till ett nytt slutsem, samt skriva en helt ny oppositionsrapport etc (om jag inte minns fel). Med tanke på effekten, borde vi kanske varna med ett meddelande. När man väljer noll poäng och klickar på submit: ”_0 points implies failing the student’s opposition report. The student will have to sign up for a new seminar and write a new opposition report. Are you sure you want to continue?_” Eller något liknande 😊 > > Känns lite out-of-scope. Lägg ett nytt ärende tycker jag. OK jag skriver ett nytt ärende. > > > 3) **Inloggad som seminarieordförande, när man skall ge poäng för oppositionsrapport.** När man tryckt ”Request Improvements” och skickat feedback till studenten. Som handledare/seminarieordförande kan man fortfarande välja att ge poäng. **Vad händer då?** Man borde frysa den möjligheten tills en ny uppdaterad version skickas av studenten. > > Fixat. Funkar bra! Tack! > > > 4) **När handledaren begärt komplettering och studenten skickat in kompletteringen.** När handledaren ger sen godkända poäng (1-2), skickas ingen (ny) notifikation till studenten? Inloggad som student kunde jag bara se notifikationen för avslag. > > Fixat. > Funkar bra! Tack! > > 5) **Inloggad som seminarieordförande, när man skall ge poäng för oppositionsrapport**. Om studenten skickar en bilaga som oppositionsrapport, syns enbart datumet för uppladdningen. Kan vi lägga till även tidpunkten för uppladdningen? Som vi gjort med final thesis-uppladdningar. > > Fixat. > Funkar bra! Tack!
andan added
po
approved
 and removed
po
needed
 labels 2025-03-05 08:27:03 +01:00
niat8586 approved these changes 2025-03-05 10:05:16 +01:00
niat8586 merged commit d008bec815 into develop 2025-03-05 10:05:38 +01:00
niat8586 deleted branch opponent-completion 2025-03-05 10:05:38 +01:00
niat8586 removed the
po
approved
 label 2025-03-05 10:08:10 +01:00
ansv7779 added the
po
approved
 label 2025-03-05 14:38:09 +01:00
Sign in to join this conversation.
No Reviewers
niat8586
No Label
po
approved
No Milestone
No project
No Assignees
4 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: DMC/scipro#78
No description provided.
Delete Branch "opponent-completion"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?