bugfix optiongroupserializer missing (#344)

* bugfix optiongroupserializer missing * small change * small change v2 * fix small stuff
django-oscar · Nov 25, 2024 · 8cf7e38 · 8cf7e38
1 parent da010c8
commit 8cf7e38
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 2 deletions.
diff --git a/Makefile b/Makefile
@@ -18,10 +18,10 @@ test:
 	python sandbox/manage.py test oscarapi --settings=sandbox.settings.block_admin_api_true
 	python sandbox/manage.py test oscarapi --settings=sandbox.settings.block_admin_api_false
 
-coverage:
-	coverage run sandbox/manage.py test oscarapi --settings=sandbox.settings.block_admin_api_true
+coverage:	
 	coverage run sandbox/manage.py test oscarapi --settings=sandbox.settings.block_admin_api_false
 	coverage report -m
+	coverage html
 	coverage xml -i
 
 docker-build:

diff --git a/oscarapi/serializers/product.py b/oscarapi/serializers/product.py
@@ -187,6 +187,7 @@ class Meta:
 
 
 class OptionSerializer(OscarHyperlinkedModelSerializer):
+    option_group = AttributeOptionGroupSerializer(required=False, allow_null=True)    
     code = serializers.SlugField()
 
     class Meta: