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

Manual user 'merge into' revealed group file error #1143

Open
torgeirl opened this issue Feb 28, 2020 · 0 comments
Open

Manual user 'merge into' revealed group file error #1143

torgeirl opened this issue Feb 28, 2020 · 0 comments
Labels

Comments

@torgeirl
Copy link
Collaborator

torgeirl commented Feb 28, 2020

Description
Awaiting a proper 'merge into' option (issue #1122) we have solved mid-semester username switches by the following procedure:

  1. Merge the old and new user on already corrected or delivered assignments.
  2. Remove the old user from assignments awaiting delivery.
  3. Deactivate the old user on active semesters to prevent it from getting added to assignments and signal to semester admins that it's not in use.

This procedure is time-consuming, error-prone, and creates additional issues:

  • the passed previous function can no longer recognize the student due to different usernames.
  • if the new user had to be manually added to the assignment before step 1 and the assignment already was corrected, the status gets masked as waiting for feedback due to issue Assignment status still masked for some assignment groups #1101.

If the old user isn't just deactivated, but split out of the group and then removed from the assignment a third issue gets introduced:

  • files uploaded by the user are still visual, but trying to download them or generating a Zip file for the assignment both causes a FileNotFoundError. Re-merging the old user to the group doesn't solve the issue.

Sticking to the old 'merge into' procedure will prevent this from happening from future username switches, but the issue still holds for other scenarios involving group changes, and we should probably handle it better than a 500 error.

Solution A

  • when an student with files is removed from ('split out of') a group an addition warning listing the file info (names, dates and size) should be displayed, warning that the files could become unavailable for the remaining students if the user later gets removed from the assignment.
  • when a student group (of 1 or more students) is removed from an assignment, also check if that effects other other student groups and if so remove their clickable link and add a strikethrough effect to them.

Solution B

  • rework how student group file ownership works to prevent this issue overall.
@torgeirl torgeirl added the bug label Feb 28, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

1 participant