Merge branch 'chore/api_endpoints' of github.com:makeplane/plane into develop-deploy

apiserver/plane/api/views/base.py

@@ -42,24 +42,6 @@ class WebhookMixin:
     def finalize_response(self, request, response, *args, **kwargs):
         response = super().finalize_response(request, response, *args, **kwargs)
 
-        # Check for the case should webhook be sent
-        if (
-            self.webhook_event
-            and self.request.method in ["POST", "PATCH"]
-            and response.status_code in [200, 201, 204]
-        ):
-            # Get the id
-            object_id = (
-                response.data.get("id") if isinstance(response.data, dict) else None
-            )
-            # Push the object to delay
-            send_webhook.delay(
-                event=self.webhook_event,
-                event_id=object_id,
-                action=self.request.method,
-                slug=self.workspace_slug,
-            )
-
         # Check for the case should webhook be sent
         if (
             self.webhook_event

apiserver/plane/api/views/cycle.py

@@ -310,7 +310,7 @@ class CycleIssueAPIEndpoint(WebhookMixin, BaseAPIView):
 
     serializer_class = CycleIssueSerializer
     model = CycleIssue
-    webhook_event = "cycle"
+    webhook_event = "cycle_issue"
     permission_classes = [
         ProjectEntityPermission,
     ]

apiserver/plane/api/views/issue.py

@@ -450,7 +450,7 @@ class IssueCommentAPIEndpoint(WebhookMixin, BaseAPIView):
 
     serializer_class = IssueCommentSerializer
     model = IssueComment
-    webhook_event = "issue-comment"
+    webhook_event = "issue_comment"
     permission_classes = [
         ProjectLitePermission,
     ]

apiserver/plane/api/views/module.py

@@ -194,7 +194,7 @@ class ModuleIssueAPIEndpoint(WebhookMixin, BaseAPIView):
 
     serializer_class = ModuleIssueSerializer
     model = ModuleIssue
-    webhook_event = "module"
+    webhook_event = "module_issue"
 
     permission_classes = [
         ProjectEntityPermission,

apiserver/plane/app/views/cycle.py

@@ -502,7 +502,7 @@ class CycleViewSet(WebhookMixin, BaseViewSet):
 class CycleIssueViewSet(WebhookMixin, BaseViewSet):
     serializer_class = CycleIssueSerializer
     model = CycleIssue
-    webhook_event = "cycle"
+    webhook_event = "cycle_issue"
     permission_classes = [
         ProjectEntityPermission,
     ]

apiserver/plane/app/views/issue.py

@@ -596,7 +596,7 @@ class IssueActivityEndpoint(BaseAPIView):
 class IssueCommentViewSet(WebhookMixin, BaseViewSet):
     serializer_class = IssueCommentSerializer
     model = IssueComment
-    webhook_event = "issue-comment"
+    webhook_event = "issue_comment"
     permission_classes = [
         ProjectLitePermission,
     ]

apiserver/plane/app/views/module.py

@@ -283,9 +283,10 @@ class ModuleViewSet(WebhookMixin, BaseViewSet):
         return Response(status=status.HTTP_204_NO_CONTENT)
 
 
-class ModuleIssueViewSet(BaseViewSet):
+class ModuleIssueViewSet(WebhookMixin, BaseViewSet):
     serializer_class = ModuleIssueSerializer
     model = ModuleIssue
+    webhook_event = "module_issue"
 
     filterset_fields = [
         "issue__labels__id",

apiserver/plane/bgtasks/analytic_plot_export.py

@@ -81,13 +81,6 @@ def send_export_email(email, slug, csv_buffer):
                 os.environ.get("EMAIL_USE_TLS", "1"),
             )
         ),
-        use_ssl=bool(
-            get_configuration_value(
-                instance_configuration,
-                "EMAIL_USE_SSL",
-                os.environ.get("EMAIL_USE_SSL", "0"),
-            )
-        ),
     )
 
     msg = EmailMultiAlternatives(

apiserver/plane/bgtasks/email_verification_task.py

@@ -65,13 +65,6 @@ def email_verification(first_name, email, token, current_site):
                     os.environ.get("EMAIL_USE_TLS", "1"),
                 )
             ),
-            use_ssl=bool(
-                get_configuration_value(
-                    instance_configuration,
-                    "EMAIL_USE_SSL",
-                    os.environ.get("EMAIL_USE_SSL", "0"),
-                )
-            ),
         )
 
         # Initiate email alternatives

apiserver/plane/bgtasks/forgot_password_task.py

@@ -60,13 +60,6 @@ def forgot_password(first_name, email, uidb64, token, current_site):
                     os.environ.get("EMAIL_USE_TLS", "1"),
                 )
             ),
-            use_ssl=bool(
-                get_configuration_value(
-                    instance_configuration,
-                    "EMAIL_USE_SSL",
-                    os.environ.get("EMAIL_USE_SSL", "0"),
-                )
-            ),
         )
 
         msg = EmailMultiAlternatives(

apiserver/plane/bgtasks/magic_link_code_task.py

@@ -59,13 +59,6 @@ def magic_link(email, key, token, current_site):
                     os.environ.get("EMAIL_USE_TLS", "1"),
                 )
             ),
-            use_ssl=bool(
-                get_configuration_value(
-                    instance_configuration,
-                    "EMAIL_USE_SSL",
-                    os.environ.get("EMAIL_USE_SSL", "0"),
-                )
-            ),
         )
 
         msg = EmailMultiAlternatives(

apiserver/plane/bgtasks/project_invitation_task.py

@@ -74,13 +74,6 @@ def project_invitation(email, project_id, token, current_site, invitor):
                     os.environ.get("EMAIL_USE_TLS", "1"),
                 )
             ),
-            use_ssl=bool(
-                get_configuration_value(
-                    instance_configuration,
-                    "EMAIL_USE_SSL",
-                    os.environ.get("EMAIL_USE_SSL", "0"),
-                )
-            ),
         )
 
         msg = EmailMultiAlternatives(

apiserver/plane/bgtasks/webhook_task.py

@@ -176,13 +176,13 @@ def send_webhook(event, event_id, action, slug):
         if event == "issue":
             webhooks = webhooks.filter(issue=True)
 
-        if event == "module":
+        if event == "module" or event == "module_issue":
             webhooks = webhooks.filter(module=True)
 
-        if event == "cycle":
+        if event == "cycle" or event == "cycle_issue":
             webhooks = webhooks.filter(cycle=True)
 
-        if event == "issue-comment":
+        if event == "issue_comment":
             webhooks = webhooks.filter(issue_comment=True)
 
         for webhook in webhooks:

apiserver/plane/bgtasks/workspace_invitation_task.py

@@ -83,13 +83,6 @@ def workspace_invitation(email, workspace_id, token, current_site, invitor):
                     os.environ.get("EMAIL_USE_TLS", "1"),
                 )
             ),
-            use_ssl=bool(
-                get_configuration_value(
-                    instance_configuration,
-                    "EMAIL_USE_SSL",
-                    os.environ.get("EMAIL_USE_SSL", "0"),
-                )
-            ),
         )
 
         msg = EmailMultiAlternatives(