Usage requirements refactor (#749)

* Added CoursePermission and CourseType

* Implemented cource permission class

* migrated data

* Updated views and admin to use new permission

* fixed bug with auth requirement

* added coursepermission to admin

* Updated course page

* Updated form ui for courses

* Made 3dpr permission automatic

* updated url tests

* update machine tests

* updated reservation test

* updated machine and reservation tests

* code cleanup

* Added translations

* Fixed api bug

* Removed print

* fixed line endings hopefully

* Fixed naming in migrations

* Fixed usage hint and translation typo

* fixed some codeclimate cleanup

* removed some whitespace

* Added changes to unreleased in changelog

* Made some fixes according to review comments

* Removed unused if statement

* Added if statement to prevent when a user has no registered course

* updated more code according to feedback

* removed whitespace

* i removed one too many whitespace lines, whoops

* fixed bug where course list used old get_permission_names method

* removed atomic flag

* merged migrations

* Switched to using enum instead of "3DPR" and similar strings

---------

Co-authored-by: Anders <6058745+ddabble@users.noreply.github.com>
Co-authored-by: Sigrid <55406589+sigtheidiot@users.noreply.github.com>
Co-authored-by: Gunvor4 <114504373+Gunvor4@users.noreply.github.com>

Author: 4 people

CHANGELOG.md

@@ -6,7 +6,7 @@ A summary of changes made to the codebase, grouped per deployment.
 
 ### New features
 
-- 
+- Made it easier to add new permissions for any machine type, so that each permission can be granted when registering a completed course
 
 ### Improvements
 

src/checkin/views.py

@@ -15,6 +15,7 @@
 from card.views import RFIDView
 from util.view_utils import PreventGetRequestsMixin
 from .models import Profile, RegisterProfile, Skill, SuggestSkill, UserSkill
+from make_queue.models.course import CoursePermission
 
 
 class AdminCheckInView(RFIDView):
@@ -107,9 +108,7 @@ def get_context_data(self, **kwargs):
         profile, _created = Profile.objects.get_or_create(user=user)
 
         completed_3d_printer = hasattr(user, 'printer_3d_course')
-        completed_raise3d = completed_3d_printer and user.printer_3d_course.raise3d_course
-        completed_sla = completed_3d_printer and user.printer_3d_course.sla_course
-
+        special_courses = CoursePermission.objects.exclude(short_name__in=[CoursePermission.DefaultPerms.TAKEN_3D_PRINTER_COURSE, CoursePermission.DefaultPerms.IS_AUTHENTICATED])
         completed_course_message_structs = [
             CompletedCourseMessageStruct(
                 completed=completed_3d_printer,
@@ -119,22 +118,18 @@ def get_context_data(self, **kwargs):
                     "To use a 3D printer, make a reservation in the calendar of one of the 3D printers on the “Reservations” page."
                 ) if completed_3d_printer else None,
             ),
-            CompletedCourseMessageStruct(
-                completed=completed_raise3d,
-                message=(_("You have completed the Raise3D printer course") if completed_raise3d
-                         else _("You have not taken the Raise3D printer course")),
-                usage_hint=_(
-                    "To use a Raise3D printer, make a reservation in the calendar of one of the Raise3D printers on the “Reservations” page."
-                ) if completed_raise3d else None,
-            ),
-            CompletedCourseMessageStruct(
-                completed=completed_sla,
-                message=(_("You have completed the SLA 3D printer course") if completed_sla
-                         else _("You have not taken the SLA 3D printer course")),
-                usage_hint=_(
-                    "To use an SLA 3D printer, make a reservation in the calendar of one of the SLA 3D printers on the “Reservations” page."
-                ) if completed_sla else None,
-            ),
+            *(
+                CompletedCourseMessageStruct(
+                    completed=user.printer_3d_course.course_permissions.filter(short_name=c.short_name).exists(),
+                    message=_("You have completed the {} course").format(c.name)
+                    if user.printer_3d_course.course_permissions.filter(short_name=c.short_name).exists()
+                    else _("You have not taken the {} course").format(c.name),
+                    usage_hint=_(
+                        "To use a {}, make a reservation in the calendar of one of the {}s on the “Reservations” page."
+                    ).format(c.name, c.name) if user.printer_3d_course.course_permissions.filter(short_name=c.short_name).exists() else None,
+                )
+                for c in special_courses if hasattr(user, 'printer_3d_course')
+            )
         ]
 
         """ Commented out because it's currently not in use; see the template code in `profile_detail_internal.html`