pr fixes

Author: opaduchak

addon_service/addon_operation_invocation/views.py

@@ -6,7 +6,6 @@
 
 from addon_service.common.permissions import (
     IsAuthenticated,
-    SessionUserIsOwner,
     SessionUserMayAccessInvocation,
     SessionUserMayPerformInvocation,
 )
@@ -52,9 +51,6 @@ def get_permissions(self):
         match self.action:
             case "retrieve" | "retrieve_related":
                 return [IsAuthenticated(), SessionUserMayAccessInvocation()]
-            case "partial_update" | "update" | "destroy":
-                # prohibit this? Maybe allow only to delete invocation from action log, but definitely not update
-                return [IsAuthenticated(), SessionUserIsOwner()]
             case "create":
                 return [SessionUserMayPerformInvocation()]
             case None:

addon_service/authorized_account/citation/views.py

@@ -11,7 +11,9 @@
 
 @extend_schema_view(
     create=extend_schema(
-        description='Create new authorized citation account for given external citation service.\n For OAuth services it\'s required to create account with `"initiate_oauth"=true` in order to proceed with OAuth flow'
+        description="Create new authorized citation account for given external citation service. "
+        'For OAuth services it\'s required to create account with `"initiate_oauth"=true` '
+        "in order to proceed with OAuth flow"
     ),
 )
 class AuthorizedCitationAccountViewSet(AuthorizedAccountViewSet):

addon_service/authorized_account/computing/views.py

@@ -11,7 +11,9 @@
 
 @extend_schema_view(
     create=extend_schema(
-        description='Create new authorized computing account for given external computing service.\n For OAuth services it\'s required to create account with `"initiate_oauth"=true` in order to proceed with OAuth flow'
+        description="Create new authorized computing account for given external computing service.\n "
+        'For OAuth services it\'s required to create account with `"initiate_oauth"=true` '
+        "in order to proceed with OAuth flow"
     ),
 )
 class AuthorizedComputingAccountViewSet(AuthorizedAccountViewSet):

addon_service/authorized_account/link/views.py

@@ -11,7 +11,9 @@
 
 @extend_schema_view(
     create=extend_schema(
-        description='Create new authorized link account for given external link service.\n For OAuth services it\'s required to create account with `"initiate_oauth"=true` in order to proceed with OAuth flow'
+        description="Create new authorized link account for given external link service.\n "
+        'For OAuth services it\'s required to create account with `"initiate_oauth"=true` '
+        "in order to proceed with OAuth flow"
     ),
 )
 class AuthorizedLinkAccountViewSet(AuthorizedAccountViewSet):

addon_service/authorized_account/storage/views.py

@@ -15,7 +15,9 @@
 
 @extend_schema_view(
     create=extend_schema(
-        description='Create new authorized storage account for given external storage service.\n For OAuth services it\'s required to create account with `"initiate_oauth"=true` in order to proceed with OAuth flow'
+        description="Create new authorized storage account for given external storage service."
+        '\n For OAuth services it\'s required to create account with `"initiate_oauth"=true` '
+        "in order to proceed with OAuth flow"
     ),
 )
 class AuthorizedStorageAccountViewSet(AuthorizedAccountViewSet):

addon_service/common/schemas.py

@@ -11,6 +11,7 @@
 from addon_service.osf_models.fields import decrypt_string
 
 
+# Exclude oAuth views from openapi schema as they are from internal use only
 @extend_schema(exclude=True)
 def oauth1_callback_view(request):
     oauth_token = request.GET["oauth_token"]

addon_service/oauth1/views.py

@@ -13,6 +13,7 @@
 from addon_service.oauth2.utils import get_initial_access_token
 
 
+# Exclude oAuth views from openapi schema as they are from internal use only
 @extend_schema(exclude=True)
 @transaction.non_atomic_requests  # async views and ATOMIC_REQUESTS do not mix
 async def oauth2_callback_view(request):