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

Updating course settings overwrites presentation score #10137

Open
maximiliansoelch opened this issue Jan 13, 2025 · 1 comment
Open

Updating course settings overwrites presentation score #10137

maximiliansoelch opened this issue Jan 13, 2025 · 1 comment
Labels
assessment Pull requests that affect the corresponding module bug

Comments

@maximiliansoelch
Copy link
Member

Describe the bug

When updating the general settings of a course, the presentation score defined in the grading scheme gets overwritten.
In the past, the presentation score was set in the course settings, but when it was moved to the grading schema, this was not properly handled and every save on the course settings now overwrites the score as it is not present in the settings' form data.

To Reproduce

  1. Go to 'Scores' -> 'Grading Key'
  2. Define a grading key and set a basic presentation mode and set the number to 2
  3. Save the grading key
  4. Go back to the course overview, click on edit
  5. On the edit view click on save at the bottom
  6. Go back to the grading key view and verify the presentation score has been overwritten

Expected behavior

When updating the course settings, the presentation score should not be overwritten

Screenshots

No response

Which version of Artemis are you seeing the problem on?

7.8.3

What browsers are you seeing the problem on?

Chrome, Safari

Additional context

No response

Relevant log output

No response

@github-actions github-actions bot added the assessment Pull requests that affect the corresponding module label Jan 13, 2025
@krusche
Copy link
Member

krusche commented Jan 13, 2025

bugfix/presentation-score includes a workaround until we have a proper fix

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
assessment Pull requests that affect the corresponding module bug
Projects
None yet
Development

No branches or pull requests

2 participants