Use 'id' instead of 'pk' in serializers

For consistency with model definitions

Author: nmdanny

Closure_Front_End/src/components/Navigation.vue

@@ -58,7 +58,7 @@ export default {
       return await fetchDjangoListIntoSelectOptions(this.http, url, track => track.name);
     },
     async fetchCourses(query) {
-      const url = `tracks/${this.selectedTrack?.pk ?? 'null'}/courses/?limit=6&offset=15&data_year=${this.selectedYear}&search=${query}`;
+      const url = `tracks/${this.selectedTrack?.id ?? 'null'}/courses/?limit=6&offset=15&data_year=${this.selectedYear}&search=${query}`;
       return await fetchDjangoListIntoSelectOptions(this.http, url, course => course.name);
     },
   }

Closure_Project/rest_api/serializers/CourseGroupSerializer.py

@@ -4,7 +4,7 @@
 
 
 class CourseGroupSerializer(DynamicFieldsModelSerializer):
-    courses = CourseSerializer(fields=('pk', 'course_id', 'name', 'semester', 'points'), many=True, read_only=True)
+    courses = CourseSerializer(fields=('id', 'course_id', 'name', 'semester', 'points'), many=True, read_only=True)
 
     class Meta:
         model = CourseGroup

Closure_Project/rest_api/serializers/CourseSerializer.py

@@ -5,12 +5,11 @@
 
 
 class CourseSerializer(DynamicFieldsModelSerializer):
-    pk = serializers.PrimaryKeyRelatedField(source='id', read_only=True)
-
     class Meta:
         model = Course
-        fields = ('pk', 'course_id', 'data_year', 'name', 'semester', 'points', 'is_given_this_year',
+        fields = ('id', 'course_id', 'data_year', 'name', 'semester', 'points', 'is_given_this_year',
                   'is_corner_stone', 'comment')
+        read_only_fields = ('id',)
 
 class CourseOfTrackSerializer(CourseSerializer):
     type = serializers.SerializerMethodField(method_name='get_type')

Closure_Project/rest_api/serializers/TrackSerializer.py

@@ -4,14 +4,14 @@
 
 
 class TrackSerializer(DynamicFieldsModelSerializer):
-    pk = serializers.PrimaryKeyRelatedField(source='id', read_only=True)
     total_points = serializers.IntegerField(read_only=True)
 
     class Meta:
         model = Track
-        fields = ('pk', 'track_number', 'name', 'data_year', 'total_points', 'points_must', 'points_from_list',
+        fields = ('id', 'track_number', 'name', 'data_year', 'total_points', 'points_must', 'points_from_list',
                   'points_choice', 'points_complementary', 'points_corner_stones',
                   'points_minor', 'points_additional_hug', 'comment')
+        read_only_fields = ('id',)
 
 class TrackSerializerWithCourseGroups(TrackSerializer):
     course_groups = CourseGroupSerializer(source='coursegroup_set', many=True, read_only=True)