Merge pull request #243 from avantifellows/embeddable

Make Plio Embeddable
avantifellows · Sep 23, 2021 · c215163 · c215163
2 parents dce5a50 + 38af7ca
commit c215163
Show file tree

Hide file tree

Showing 3 changed files with 12 additions and 4 deletions.
diff --git a/organizations/serializers.py b/organizations/serializers.py
@@ -5,5 +5,13 @@
 class OrganizationSerializer(serializers.ModelSerializer):
     class Meta:
         model = Organization
-        fields = ["id", "schema_name", "name", "shortcode", "created_at", "updated_at"]
+        fields = [
+            "id",
+            "schema_name",
+            "name",
+            "shortcode",
+            "api_key",
+            "created_at",
+            "updated_at",
+        ]
         read_only_fields = ["schema_name"]
diff --git a/plio/tests.py b/plio/tests.py
@@ -232,12 +232,12 @@ def test_user_list_plio_uuids(self):
             },
         )
 
-    def test_guest_cannot_play_plio(self):
+    def test_guest_can_play_plio(self):
         # unset the credentials
         self.client.credentials()
         # play plio
         response = self.client.get(f"/api/v1/plios/{self.plio_1.uuid}/play/")
-        self.assertEqual(response.status_code, status.HTTP_401_UNAUTHORIZED)
+        self.assertEqual(response.status_code, status.HTTP_200_OK)
 
     def test_user_can_play_own_plio(self):
         # play plio

diff --git a/plio/views.py b/plio/views.py
@@ -170,7 +170,7 @@ def list_uuid(self, request):
     @action(
         methods=["get"],
         detail=True,
-        permission_classes=[IsAuthenticated, PlioPermission],
+        permission_classes=[PlioPermission],
     )
     def play(self, request, uuid):
         queryset = Plio.objects.filter(uuid=uuid)