address review - survive non-existing project version

Author: harminius

server/mergin/sync/public_api_v2_controller.py

@@ -139,9 +139,10 @@ def get_project(id, files_at_version=None):
     if files_at_version:
         pv = ProjectVersion.query.filter_by(
             project_id=project.id, name=ProjectVersion.from_v_name(files_at_version)
-        ).first_or_404()
-        data["files"] = ProjectFileSchema(
-            only=("path", "mtime", "size", "checksum"), many=True
-        ).dump(pv.files)
+        ).first()
+        if pv:
+            data["files"] = ProjectFileSchema(
+                only=("path", "mtime", "size", "checksum"), many=True
+            ).dump(pv.files)
 
     return data, 200

server/mergin/sync/schemas_v2.py

@@ -30,9 +30,7 @@ class ProjectSchema(ma.SQLAlchemyAutoSchema):
 
     def _role(self, obj):
         role = ProjectPermissions.get_user_project_role(obj, current_user)
-        if not role:
-            return None
-        return role.value
+        return role.value if role else None
 
     class Meta:
         model = Project