Merge pull request #3443 from ngokevin/commcors

fix note list cors
mozilla · Nov 16, 2015 · 3d9d8b3 · 3d9d8b3
2 parents 899e677 + 8d775a8
commit 3d9d8b3
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 1 deletion.
diff --git a/mkt/comm/tests/test_views.py b/mkt/comm/tests/test_views.py
@@ -473,6 +473,10 @@ def _grant_perms(self):
         self.grant_permission(self.profile, 'Apps:Review')
         self.grant_permission(self.profile, 'ContentTools:AddonReview')
 
+    def test_cors(self):
+        res = self.client.get(self.url)
+        self.assertCORS(res, 'get')
+
     def test_no_perm(self):
         res = self.client.get(self.url)
         eq_(res.status_code, 403)

diff --git a/mkt/comm/views.py b/mkt/comm/views.py
@@ -192,7 +192,7 @@ def create(self, request, *args, **kwargs):
             status=status.HTTP_201_CREATED)
 
 
-class NoteListView(ListAPIView, MarketplaceView):
+class NoteListView(ListAPIView, MarketplaceView, CORSMixin):
     serializer_class = NoteSerializer
     authentication_classes = [RestOAuthAuthentication,
                               RestSharedSecretAuthentication]