Revert "Feat: Add a additional language in the courserun API model. (#89

)" This reverts commit 30b73a3.
edly-io · Sep 10, 2024 · 70b17b0 · 70b17b0
1 parent 30b73a3
commit 70b17b0
Show file tree

Hide file tree

Showing 6 changed files with 1 addition and 36 deletions.
diff --git a/course_discovery/apps/api/serializers.py b/course_discovery/apps/api/serializers.py
@@ -930,7 +930,7 @@ class Meta(MinimalCourseRunSerializer.Meta):
             'course_uuid', 'estimated_hours', 'invite_only', 'subjects',
             'is_marketing_price_set', 'marketing_price_value', 'is_marketing_price_hidden', 'featured', 'card_image_url',
             'average_rating', 'total_raters', 'yt_video_url', 'course_duration_override', 'course_difficulty',
-            'course_job_role', 'course_format', 'course_industry_certified_training', 'course_owner', 'course_language'
+            'course_job_role', 'course_format', 'course_industry_certified_training', 'course_owner'
         )
         read_only_fields = ('enrollment_count', 'recent_enrollment_count',)
 
@@ -2278,7 +2278,6 @@ class Meta:
             'course_format',
             'course_industry_certified_training',
             'course_owner',
-            'course_language'
         )
 
 

diff --git a/course_discovery/apps/api/tests/test_serializers.py b/course_discovery/apps/api/tests/test_serializers.py
@@ -660,7 +660,6 @@ def get_expected_data(cls, course_run, request):
             'is_marketing_price_hidden': course_run.is_marketing_price_hidden,
             'card_image_url': course_run.card_image_url,
             'subjects': [],
-            'course_language': course_run.course_language
         })
         return expected
 

diff --git a/course_discovery/apps/course_metadata/data_loaders/api.py b/course_discovery/apps/course_metadata/data_loaders/api.py
@@ -1083,7 +1083,6 @@ def _process_response(self, response):
                 course_run.course_job_role = body['course_job_role']
                 course_run.course_format = body['course_format']
                 course_run.course_industry_certified_training = body['course_industry_certified_training']
-                course_run.course_language = body['course_language']
                 course_run.course_owner = body['course_owner']
                 course_run.status = self._process_course_status(body['status'])
 

diff --git a/course_discovery/apps/course_metadata/migrations/0272_auto_20240830_1141.py b/course_discovery/apps/course_metadata/migrations/0272_auto_20240830_1141.py
diff --git a/course_discovery/apps/course_metadata/models.py b/course_discovery/apps/course_metadata/models.py
@@ -1346,13 +1346,6 @@ class CourseRun(DraftModelMixin, CachedMixin, TimeStampedModel):
         max_length=255, null=True, blank=True, verbose_name=_("Course Owner")
     )
 
-    course_language = models.CharField(
-        max_length=255,
-        null=True,
-        blank=True,
-        verbose_name=_("Language of the course run"),
-    )
-
     STATUS_CHANGE_EXEMPT_FIELDS = [
         'start',
         'end',

diff --git a/course_discovery/apps/course_metadata/search_indexes.py b/course_discovery/apps/course_metadata/search_indexes.py
@@ -258,8 +258,6 @@ class CourseRunIndex(BaseCourseIndex, indexes.Indexable):
     course_job_role = indexes.CharField(model_attr='course_job_role', null=True)
     course_format = indexes.CharField(model_attr='course_format', null=True)
     course_industry_certified_training = indexes.CharField(model_attr='course_industry_certified_training', null=True)
-    course_language = indexes.CharField(model_attr='course_language', null=True)
-
     course_owner = indexes.CharField(model_attr='course_owner', null=True)
 
     def read_queryset(self, using=None):