Allow supervisors to request improvements from final seminar opponents #78

Open

ansv7779 wants to merge 40 commits from opponent-completion into develop

pull from: opponent-completion

merge into: DMC:develop

Conversation 6 Commits 40 Files Changed 53 +1153 -263

ansv7779 commented 2025-01-13 16:28:56 +01:00

Owner

Copy Link

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/367/jobs/0 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/367/jobs/0 to reset the database. It takes a few minutes.

ansv7779 added 2 commits 2025-01-13 16:28:57 +01:00

GUI to request improvements 9d88c373f2

Create test data 8ce045b33c

gitea-actions commented 2025-01-13 16:32:42 +01:00

First-time contributor

Copy Link

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

foo 3c83bdd067

Opposition grading is now based on the opponents grading report criteria 257536d856

Refactored away business logic from the Wicket panel ... 5c0238485e

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

Use the same project type for opponent and seminar c4febb11e7

Verify that opposition criteria are based on the grading report template 46cc99972f

Implement gradeOpponent 49f14a00f0

ansv7779 added 2 commits 2025-01-17 13:19:43 +01:00

Fix test issues with multiple EventBus getting registered db38a70675

Fix compilation problem 54e6e5d787

ansv7779 added 1 commit 2025-01-17 15:18:41 +01:00

Save assessment fd01796121

ansv7779 added 1 commit 2025-01-17 15:21:21 +01:00

Fix test 7a3b8d4a98

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

Fix test ecdd68efbe

ansv7779 added 1 commit 2025-01-21 14:55:06 +01:00

Replace direct repository access with a service da91d4074d

ansv7779 added 1 commit 2025-01-23 17:47:00 +01:00

Replace @Entity with DTO fec631f2d0

ansv7779 added 1 commit 2025-01-23 21:17:44 +01:00

Show requests for improvement when filling in the opposition report 54d5ef8752

ansv7779 added 1 commit 2025-01-24 15:16:08 +01:00

Translated some swedish text into english. ... 9db30b3d28

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

Notify opponent when improvements are requested 88ced02a2d

ansv7779 added 2 commits 2025-02-06 16:38:24 +01:00

Fail authors who do not improve their opposition report in time. 08302f9719

Switch back to the grading form after requesting improvements 34e986a6c9

ansv7779 added 3 commits 2025-02-07 12:40:34 +01:00

Move worker @Beans to the correct @Configuration class 9823dc193c

Extract common functionality for both branches 5e0d285272

Do not remind supervisors to grade opponents if they've requested improvements 305142a862

ansv7779 added 1 commit 2025-02-07 15:28:16 +01:00

Show information to the supervisor that they've requested improvements 791ec65633

ansv7779 added 2 commits 2025-02-10 10:48:59 +01:00

Remove the AttachmentReport abstraction on the panel to fill out opposition report ... 1299af5b00

The reviewer report has long since been removed and the supervisors report differs too much and had been decoupled already.

Send out notification when an opponent submits a report 06c1e81978

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

Merge branch 'develop' into opponent-completion a20455df97

niat8586 requested review from niat8586 2025-02-11 08:53:35 +01:00

niat8586 commented 2025-02-11 10:07:48 +01:00

Owner

Copy Link

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.

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.  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

Better UX for notification about opposition report needing improvements ... 236f6a0ad3

Now takes the opponent directly to the form where they can fill in the report and view the feedback from the seminar supervisor.

Alert the opponent that they need to an improved report on the final seminar page 3be41635b1

ansv7779 added 1 commit 2025-02-11 15:47:26 +01:00

Styling improvements 87e60578c0

ansv7779 commented 2025-02-11 15:48:39 +01:00

Author

Owner

Copy Link

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

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. >  > > 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

Merge branch 'develop' into opponent-completion ... 6410710679

# Conflicts:
#	core/src/main/java/se/su/dsv/scipro/DataInitializer.java

ansv7779 added 1 commit 2025-02-12 13:45:54 +01:00

Merge branch 'develop' into opponent-completion 8892bf4780

niat8586 approved these changes 2025-02-12 15:10:23 +01:00

niat8586 left a comment

Owner

Copy Link

👍

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

Copy Link

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

Merge branch 'develop' into opponent-completion ... 970f6ee61b

# Conflicts:
#	core/src/main/java/se/su/dsv/scipro/DataInitializer.java

ansv7779 added 6 commits 2025-02-18 15:19:29 +01:00

Enable *all* notifications during development and on test servers cfef02b98c

Send notifications to opponents when they're approved 093dd11c29

Show the time that opponents submitted their report cbd426be07

Remove unused dependencies ad91a9b961

Disable ability to grade opponents after requesting improvements 1b36ad7987

Remove useless PhD project type ... d292ecc34d

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

Copy Link

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.

> 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

Merge branch 'develop' into opponent-completion ... d0fb2e721c

# Conflicts:
#	core/src/main/java/se/su/dsv/scipro/DataInitializer.java

All checks were successful

Hide all checks

Deploy to branch.dsv.su.se / deploy (pull_request) Successful in 3m30s

Details

Build and test / build-and-test (push) Successful in 19m58s

Details

This pull request can be merged automatically.

You are not authorized to merge this pull request.

View command line instructions.

Checkout

From your project repository, check out a new branch and test the changes.

git fetch -u origin opponent-completion:opponent-completion

git checkout opponent-completion

Sign in to join this conversation.

Reviewers

No Reviewers

niat8586

Labels

Clear labels

bug

Something is not working

discussion

duplicate

This issue or pull request already exists

enhancement

New feature

frozen

The issue is not solved, but it will be investigated in the future it the issue arises again

help wanted

Need some help

invalid

Something is wrong

new

To be discussed at the next meeting

po

approved

po

changes

PO has requested changes

po

needed

question

More information is needed

size

1

size

2

size

3

wontfix

This won't be fixed

No Label

po

needed

Milestone

No items

No Milestone

Projects

Clear projects

No project

Assignees

Clear assignees

No Assignees

4 Participants

Notifications

Subscribe

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: DMC/scipro#78

No description provided.