ls1intum / Artemis

Artemis - Interactive Learning with Automated Feedback
https://docs.artemis.cit.tum.de
MIT License
486 stars 288 forks source link

`Exam mode`: Importing a user with incorrespond registration number and login account failed without error report to user #7570

Closed RY997 closed 3 months ago

RY997 commented 10 months ago

Describe the bug

When importing users whose registration numbers and login accounts belong to different users, the import is displayed successfully, but it is not displayed on the main page. An error such as "User information does not correspond" should be reported.

To Reproduce

  1. Go to an existing exam or create a new one
  2. Click on "Students" and then "Import students"
  3. Upload a CSV file and import it

Expected behavior

When the registration number and login account are inconsistent but valid, we should have an error report.

Screenshots

image-2023-11-10_10-54-55

image-2023-11-10_10-55-16

Which version of Artemis are you seeing the problem on?

6.6.4

What browsers are you seeing the problem on?

Chrome

Additional context

No response

Relevant log output

No response

sarpsahinalp commented 4 months ago

Could not reproduce the issue. It seems that this happens only when the login belongs to an instructor in the course.

florian-glombik commented 3 months ago

I think that @sarpsahinalp is right and it is only related to instructors: Exam mode: Missing instructor validation on csv user import #8879