Skip to content

1774 change_appointment permission

Bengfort requested to merge 1774-book-appointments-perm into main

This adds a new permission: change_appointment. Both recruiters and study conductors have this permission by default. But it allows as to have users without that specific permissions as well as sharing some views that are not specific to either recruitment or execution (i.e. the calendar view).

Design: I was not completely sure what should be covered by this new permission. In recruitment it is now required for appointments, i.e. you cannot even see appointments without the permission. In execution it is only required for changing appointments. You can still see appointments without the permission as well as setting showup. @goettel is that what you intended?

RFC: The ContactView gets evermore complex: Now you can only edit parts of the form if you have a special permission. I implemented this by using different form classes based on permission. ContactAndAppointmentsForm simply inherits from both forms. I have never seen this approach before but it seems to work.

Merge request reports