Add query filter for dataview viewset

onadata/apps/api/tests/viewsets/test_dataview_viewset.py

@@ -454,6 +454,16 @@ def test_list_dataview(self):
         self.assertEqual(response.status_code, 200)
         self.assertEqual(len(response.data), 2)
 
+        # delete DataView and check that we don't get it in response
+        dataview = DataView.objects.get(name="My DataView2")
+        deleted_dataview_id = dataview.id
+        dataview.soft_delete(user=self.user)
+        response = view(request)
+
+        self.assertEqual(response.status_code, 200)
+        self.assertEqual(len(response.data), 1)
+        self.assertNotEqual(response.data[0]["dataviewid"], deleted_dataview_id)
+
         anon_request = request = self.factory.get("/")
         anon_response = view(anon_request)
         self.assertEqual(anon_response.status_code, 401)

onadata/apps/api/viewsets/dataview_viewset.py

@@ -155,6 +155,14 @@ def get_serializer_class(self):
 
         return serializer_class
 
+    def filter_queryset(self, queryset):
+        """
+        Remove deleted DataViews
+        """
+        return super().filter_queryset(
+            queryset.filter(deleted_at=None, deleted_by=None)
+        )
+
     def list(self, request, *args, **kwargs):
         """
         List endpoint for Filtered datasets

onadata/libs/serializers/dataview_serializer.py

@@ -109,11 +109,6 @@ class DataViewSerializer(serializers.HyperlinkedModelSerializer):
     project = serializers.HyperlinkedRelatedField(
         view_name="project-detail", lookup_field="pk", queryset=Project.objects.all()
     )
-    deleted_by = serializers.HyperlinkedRelatedField(
-        view_name="user-detail",
-        lookup_field="username",
-        read_only=True,
-    )
     columns = JsonField()
     query = JsonField(required=False)
     count = serializers.SerializerMethodField()
@@ -124,7 +119,6 @@ class DataViewSerializer(serializers.HyperlinkedModelSerializer):
 
     class Meta:
         model = DataView
-        exclude = ["deleted_by", "deleted_at"]
         fields = (
             "dataviewid",
             "name",