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

Correctors should be able to create students in TMS #1015

Closed
NadineNKoch opened this issue Oct 21, 2022 · 8 comments · Fixed by #1060
Closed

Correctors should be able to create students in TMS #1015

NadineNKoch opened this issue Oct 21, 2022 · 8 comments · Fixed by #1060
Labels

Comments

@NadineNKoch
Copy link

🎫 Description

The correctors should be able to also create student in TMS.

@NadineNKoch NadineNKoch added the 🌟 enhancement New feature or request label Oct 21, 2022
@Dudrie
Copy link
Collaborator

Dudrie commented Oct 21, 2022

The question I have is: Why should correctors be able to create students?

@NadineNKoch
Copy link
Author

That we employees do not have to do this. This reduces our workload and it is fairer if the correctors as the tutors create their students.

@Dudrie
Copy link
Collaborator

Dudrie commented Oct 21, 2022

But employees should not do this in the first place. Tutors should create their students. Do you have tutorials without a designated tutor?

@NadineNKoch
Copy link
Author

There are several tutorials that are lead by employees. The exercises from those groups are corrected by correctors. For the employees, it would be great to reduce the workload and enable their correctors to set up students in TMS.

@Dudrie
Copy link
Collaborator

Dudrie commented Oct 21, 2022

Would setting the correctors as tutors be an option instead?

@Dudrie
Copy link
Collaborator

Dudrie commented Oct 22, 2022

Alternative idea: If #1012 would be implemented in a way that tutors (not correctors) can import CSV files the correctors could be tasked to create such a CSV file. This CSV file can imported by the actual tutor afterwards. This would keep the role restrictions as is and correctors would not be able to access more information than they need.

@nk-coding
Copy link
Collaborator

Setting the corrector as tutor does not seem to be an option at least to me, as it makes it impossible for the real tutor to edit/see necessary information.
Giving correctors the permission to edit student data also does not seem to be a good option, as it makes the corrector role basically pointless (if tutors and correctors can do the same, why having the distinction in the first place).
For now, the csv import seems to be a good option, as it allows corrector to do the majority of the work, however keeps this information private in general.
For the future, maybe allowing multiple tutors may be an option, in the case the corrector should also do tutor-exclusive stuff

@Dudrie Dudrie mentioned this issue Nov 7, 2022
2 tasks
@nk-coding
Copy link
Collaborator

Decision: we want the "multiple tutors" feature

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants