Skip to content

Conversation

florian-jaeger
Copy link
Contributor

No description provided.

natkam and others added 8 commits August 5, 2025 19:51
* fix: Improve logging when feedback generation fails

* chore: Remove an unused dataclass, fix some typos and type annotations

* fix: Fix a potential security issue when running Git commands in a subprocess

When running multiple commands with `sh -c ...`, it would be easier to construct
the variable "message" so that it contained some executable code.

* fix: Fix a typo in the log message after a successful autograde job

* fix: Do not try to generate feedback for notebooks created by students

If the instructor specifies '*.ipynb' in the whitelist of an assignment,
additional notebooks created by students are also copied over. The feedback
generator should not try to generate feedback for them - they are not in
the gradebook and an attempt to do so would fail.

Refs: #272

* fix: Improve error handling and logging in GitBaseHandler.git_reponse()
* refactor: changed load_roles dict entries to lists

* chore: added new load_roles style to dev env

* Update a docstring

---------

Co-authored-by: Natalia Maniakowska <3858688+natkam@users.noreply.github.com>
* refactor: Small improvements related to submission handling

* fix: Fix soft-deleting a submission

Now:
- double delete does NOT remove the submission from the db; it just has the state `deleted`;
- if the deadline is not set, it is possible to delete a submission;
- a student can only delete own submission.

* fix: Make `SubmissionObjectHandler.delete()` async

* chore: Write tests for the delete submission endpoint

* fix: Fix some invalid feedback status values in submission handlers tests

* chore: Update the docstring for submission's delete
@florian-jaeger florian-jaeger merged commit c5a98c9 into main Aug 22, 2025
16 checks passed
@florian-jaeger florian-jaeger deleted the release-0.8.2 branch August 22, 2025 09:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants