Merge pull request #1036 from opengisch/patch_project

Improve the project patch endpoint

docker-app/qfieldcloud/core/serializers.py

@@ -101,6 +101,18 @@ class Meta:
             "user_role",
             "user_role_origin",
         )
+        read_only_fields = (
+            "private",
+            "created_at",
+            "updated_at",
+            "data_last_packaged_at",
+            "data_last_updated_at",
+            "can_repackage",
+            "needs_repackaging",
+            "status",
+            "user_role",
+            "user_role_origin",
+        )
         model = Project
 
 

docker-app/qfieldcloud/core/views/projects_views.py

@@ -97,7 +97,7 @@ def get_queryset(self):
         return projects
 
     @transaction.atomic
-    def perform_update(self, serializer):
+    def perform_update(self, serializer: ProjectSerializer) -> None:
         # Here we do an additional check if the owner has changed. If so, the reciever
         # of the project must have enough storage quota, otherwise the transfer is
         # not permitted.
@@ -111,7 +111,7 @@ def perform_update(self, serializer):
 
         # If owner has not changed, no additional check is made
         if old_owner == new_owner:
-            return
+            return None
 
         # Owner has changed, we must ensure he has enough quota for that
         # (in this transaction, the project is his already, so we just need to