remove reviewer-model

It could make some things easier, if there is no special reviewer-model, which is in fact a tutor with special privileges, but a boolean-variable in tutor "is_reviewer".

Tutors and reviewers are wildly distinct! It is true that the models are quite alike, but it doesn’t hurt at all and makes it easier to customize each user group in the future without any refactoring steps. And that will happen.

Also the permission handling relies on these models.

Additionally it would break uniformity regarding the student model which already established a way of customizing user accounts.

Hope that sounds reasonable, so please refrain from refactoring these models.

closed

reopened

removed Needs discussion label

added Needs discussion label

Should be solved by #49 (closed).

removed priority:low label

added Needs assignment and removed Needs discussion labels

mentioned in commit e9de903c

mentioned in commit ce91bfda

closed via commit ce91bfda

closed via merge request !30 (merged)