chore: code cleanup

apiserver/plane/app/serializers/project.py

@@ -44,16 +44,12 @@ class ProjectSerializer(BaseSerializer):
                 detail="Project Identifier is taken"
             )
         project = Project.objects.create(
-            **validated_data,
-            workspace_id=self.context["workspace_id"],
+            **validated_data, workspace_id=self.context["workspace_id"]
         )
         _ = ProjectIdentifier.objects.create(
             name=project.identifier,
             project=project,
             workspace_id=self.context["workspace_id"],
-            created_by_id=project.created_by_id,
-            updated_by_id=project.updated_by_id,
-            current_active=project.identifier,
         )
         return project
 
@@ -65,38 +61,29 @@ class ProjectSerializer(BaseSerializer):
             project = super().update(instance, validated_data)
             return project
 
-        # When updating check if the project identifier is different
-        if instance.identifier == identifier:
+        # If no Project Identifier is found create it
+        project_identifier = ProjectIdentifier.objects.filter(
+            name=identifier, workspace_id=instance.workspace_id
+        ).first()
+        if project_identifier is None:
+            project = super().update(instance, validated_data)
+            project_identifier = ProjectIdentifier.objects.filter(
+                project=project
+            ).first()
+            if project_identifier is not None:
+                project_identifier.name = identifier
+                project_identifier.save()
+            return project
+        # If found check if the project_id to be updated and identifier project id is same
+        if project_identifier.project_id == instance.id:
+            # If same pass update
             project = super().update(instance, validated_data)
             return project
 
-        # Check if this project identifier is already taken in the current workspace
-        if ProjectIdentifier.objects.filter(
-            name=identifier,
-            workspace_id=instance.workspace_id,
-        ).exists():
-            raise serializers.ValidationError(
-                detail="Project Identifier is already taken"
-            )
-
-        # Update the project identifier
-        project = super().update(instance, validated_data)
-        # If no Project Identifier is found create it and deactivate the old ones
-        ProjectIdentifier.objects.filter(
-            project_id=instance.id,
-            workspace_id=instance.workspace_id,
-        ).update(current_active=identifier)
-        # Create new project identifier
-        ProjectIdentifier.objects.create(
-            name=identifier,
-            project=instance,
-            workspace_id=instance.workspace_id,
-            created_by_id=project.created_by_id,
-            updated_by_id=project.updated_by_id,
-            current_active=identifier,
+        # If not same fail update
+        raise serializers.ValidationError(
+            detail="Project Identifier is already taken"
         )
-        # Return the updated project
-        return project
 
 
 class ProjectLiteSerializer(BaseSerializer):

apiserver/plane/app/urls/cycle.py

@@ -14,7 +14,7 @@ from plane.app.views import (
 
 urlpatterns = [
     path(
-        "workspaces/<str:slug>/projects/<str:project_id>/cycles/",
+        "workspaces/<str:slug>/projects/<uuid:project_id>/cycles/",
         CycleViewSet.as_view(
             {
                 "get": "list",
@@ -24,7 +24,7 @@ urlpatterns = [
         name="project-cycle",
     ),
     path(
-        "workspaces/<str:slug>/projects/<str:project_id>/cycles/<uuid:pk>/",
+        "workspaces/<str:slug>/projects/<uuid:project_id>/cycles/<uuid:pk>/",
         CycleViewSet.as_view(
             {
                 "get": "retrieve",
@@ -36,7 +36,7 @@ urlpatterns = [
         name="project-cycle",
     ),
     path(
-        "workspaces/<str:slug>/projects/<str:project_id>/cycles/<uuid:cycle_id>/cycle-issues/",
+        "workspaces/<str:slug>/projects/<uuid:project_id>/cycles/<uuid:cycle_id>/cycle-issues/",
         CycleIssueViewSet.as_view(
             {
                 "get": "list",
@@ -46,7 +46,7 @@ urlpatterns = [
         name="project-issue-cycle",
     ),
     path(
-        "workspaces/<str:slug>/projects/<str:project_id>/cycles/<uuid:cycle_id>/cycle-issues/<uuid:issue_id>/",
+        "workspaces/<str:slug>/projects/<uuid:project_id>/cycles/<uuid:cycle_id>/cycle-issues/<uuid:issue_id>/",
         CycleIssueViewSet.as_view(
             {
                 "get": "retrieve",
@@ -58,12 +58,12 @@ urlpatterns = [
         name="project-issue-cycle",
     ),
     path(
-        "workspaces/<str:slug>/projects/<str:project_id>/cycles/date-check/",
+        "workspaces/<str:slug>/projects/<uuid:project_id>/cycles/date-check/",
         CycleDateCheckEndpoint.as_view(),
         name="project-cycle-date",
     ),
     path(
-        "workspaces/<str:slug>/projects/<str:project_id>/user-favorite-cycles/",
+        "workspaces/<str:slug>/projects/<uuid:project_id>/user-favorite-cycles/",
         CycleFavoriteViewSet.as_view(
             {
                 "get": "list",
@@ -73,7 +73,7 @@ urlpatterns = [
         name="user-favorite-cycle",
     ),
     path(
-        "workspaces/<str:slug>/projects/<str:project_id>/user-favorite-cycles/<uuid:cycle_id>/",
+        "workspaces/<str:slug>/projects/<uuid:project_id>/user-favorite-cycles/<uuid:cycle_id>/",
         CycleFavoriteViewSet.as_view(
             {
                 "delete": "destroy",
@@ -82,27 +82,27 @@ urlpatterns = [
         name="user-favorite-cycle",
     ),
     path(
-        "workspaces/<str:slug>/projects/<str:project_id>/cycles/<uuid:cycle_id>/transfer-issues/",
+        "workspaces/<str:slug>/projects/<uuid:project_id>/cycles/<uuid:cycle_id>/transfer-issues/",
         TransferCycleIssueEndpoint.as_view(),
         name="transfer-issues",
     ),
     path(
-        "workspaces/<str:slug>/projects/<str:project_id>/cycles/<uuid:cycle_id>/user-properties/",
+        "workspaces/<str:slug>/projects/<uuid:project_id>/cycles/<uuid:cycle_id>/user-properties/",
         CycleUserPropertiesEndpoint.as_view(),
         name="cycle-user-filters",
     ),
     path(
-        "workspaces/<str:slug>/projects/<str:project_id>/cycles/<uuid:cycle_id>/archive/",
+        "workspaces/<str:slug>/projects/<uuid:project_id>/cycles/<uuid:cycle_id>/archive/",
         CycleArchiveUnarchiveEndpoint.as_view(),
         name="cycle-archive-unarchive",
     ),
     path(
-        "workspaces/<str:slug>/projects/<str:project_id>/archived-cycles/",
+        "workspaces/<str:slug>/projects/<uuid:project_id>/archived-cycles/",
         CycleArchiveUnarchiveEndpoint.as_view(),
         name="cycle-archive-unarchive",
     ),
     path(
-        "workspaces/<str:slug>/projects/<str:project_id>/archived-cycles/<uuid:pk>/",
+        "workspaces/<str:slug>/projects/<uuid:project_id>/archived-cycles/<uuid:pk>/",
         CycleArchiveUnarchiveEndpoint.as_view(),
         name="cycle-archive-unarchive",
     ),

apiserver/plane/app/urls/estimate.py

@@ -10,12 +10,12 @@ from plane.app.views import (
 
 urlpatterns = [
     path(
-        "workspaces/<str:slug>/projects/<str:project_id>/project-estimates/",
+        "workspaces/<str:slug>/projects/<uuid:project_id>/project-estimates/",
         ProjectEstimatePointEndpoint.as_view(),
         name="project-estimate-points",
     ),
     path(
-        "workspaces/<str:slug>/projects/<str:project_id>/estimates/",
+        "workspaces/<str:slug>/projects/<uuid:project_id>/estimates/",
         BulkEstimatePointEndpoint.as_view(
             {
                 "get": "list",
@@ -25,7 +25,7 @@ urlpatterns = [
         name="bulk-create-estimate-points",
     ),
     path(
-        "workspaces/<str:slug>/projects/<str:project_id>/estimates/<uuid:estimate_id>/",
+        "workspaces/<str:slug>/projects/<uuid:project_id>/estimates/<uuid:estimate_id>/",
         BulkEstimatePointEndpoint.as_view(
             {
                 "get": "retrieve",
@@ -36,7 +36,7 @@ urlpatterns = [
         name="bulk-create-estimate-points",
     ),
     path(
-        "workspaces/<str:slug>/projects/<str:project_id>/estimates/<uuid:estimate_id>/estimate-points/",
+        "workspaces/<str:slug>/projects/<uuid:project_id>/estimates/<uuid:estimate_id>/estimate-points/",
         EstimatePointEndpoint.as_view(
             {
                 "post": "create",
@@ -45,7 +45,7 @@ urlpatterns = [
         name="estimate-points",
     ),
     path(
-        "workspaces/<str:slug>/projects/<str:project_id>/estimates/<uuid:estimate_id>/estimate-points/<estimate_point_id>/",
+        "workspaces/<str:slug>/projects/<uuid:project_id>/estimates/<uuid:estimate_id>/estimate-points/<estimate_point_id>/",
         EstimatePointEndpoint.as_view(
             {
                 "patch": "partial_update",

apiserver/plane/app/urls/external.py

@@ -2,10 +2,7 @@ from django.urls import path
 
 
 from plane.app.views import UnsplashEndpoint
-from plane.app.views import (
-    GPTIntegrationEndpoint,
-    WorkspaceGPTIntegrationEndpoint,
-)
+from plane.app.views import GPTIntegrationEndpoint, WorkspaceGPTIntegrationEndpoint
 
 
 urlpatterns = [
@@ -15,11 +12,11 @@ urlpatterns = [
         name="unsplash",
     ),
     path(
-        "workspaces/<str:slug>/projects/<str:project_id>/ai-assistant/",
+        "workspaces/<str:slug>/projects/<uuid:project_id>/ai-assistant/",
         GPTIntegrationEndpoint.as_view(),
         name="importer",
     ),
-    path(
+   path(
         "workspaces/<str:slug>/ai-assistant/",
         WorkspaceGPTIntegrationEndpoint.as_view(),
         name="importer",

apiserver/plane/app/urls/inbox.py

@@ -9,7 +9,7 @@ from plane.app.views import (
 
 urlpatterns = [
     path(
-        "workspaces/<str:slug>/projects/<str:project_id>/inboxes/",
+        "workspaces/<str:slug>/projects/<uuid:project_id>/inboxes/",
         InboxViewSet.as_view(
             {
                 "get": "list",
@@ -19,7 +19,7 @@ urlpatterns = [
         name="inbox",
     ),
     path(
-        "workspaces/<str:slug>/projects/<str:project_id>/inboxes/<uuid:pk>/",
+        "workspaces/<str:slug>/projects/<uuid:project_id>/inboxes/<uuid:pk>/",
         InboxViewSet.as_view(
             {
                 "get": "retrieve",
@@ -30,7 +30,7 @@ urlpatterns = [
         name="inbox",
     ),
     path(
-        "workspaces/<str:slug>/projects/<str:project_id>/inbox-issues/",
+        "workspaces/<str:slug>/projects/<uuid:project_id>/inbox-issues/",
         InboxIssueViewSet.as_view(
             {
                 "get": "list",
@@ -40,7 +40,7 @@ urlpatterns = [
         name="inbox-issue",
     ),
     path(
-        "workspaces/<str:slug>/projects/<str:project_id>/inbox-issues/<uuid:issue_id>/",
+        "workspaces/<str:slug>/projects/<uuid:project_id>/inbox-issues/<uuid:issue_id>/",
         InboxIssueViewSet.as_view(
             {
                 "get": "retrieve",

apiserver/plane/app/urls/issue.py

@@ -19,18 +19,17 @@ from plane.app.views import (
     IssueUserDisplayPropertyEndpoint,
     IssueViewSet,
     LabelViewSet,
-    BulkIssueOperationsEndpoint,
     BulkArchiveIssuesEndpoint,
 )
 
 urlpatterns = [
     path(
-        "workspaces/<str:slug>/projects/<str:project_id>/issues/list/",
+        "workspaces/<str:slug>/projects/<uuid:project_id>/issues/list/",
         IssueListEndpoint.as_view(),
         name="project-issue",
     ),
     path(
-        "workspaces/<str:slug>/projects/<str:project_id>/issues/",
+        "workspaces/<str:slug>/projects/<uuid:project_id>/issues/",
         IssueViewSet.as_view(
             {
                 "get": "list",
@@ -40,7 +39,7 @@ urlpatterns = [
         name="project-issue",
     ),
     path(
-        "workspaces/<str:slug>/projects/<str:project_id>/issues/<str:pk>/",
+        "workspaces/<str:slug>/projects/<uuid:project_id>/issues/<uuid:pk>/",
         IssueViewSet.as_view(
             {
                 "get": "retrieve",
@@ -52,7 +51,7 @@ urlpatterns = [
         name="project-issue",
     ),
     path(
-        "workspaces/<str:slug>/projects/<str:project_id>/issue-labels/",
+        "workspaces/<str:slug>/projects/<uuid:project_id>/issue-labels/",
         LabelViewSet.as_view(
             {
                 "get": "list",
@@ -62,7 +61,7 @@ urlpatterns = [
         name="project-issue-labels",
     ),
     path(
-        "workspaces/<str:slug>/projects/<str:project_id>/issue-labels/<uuid:pk>/",
+        "workspaces/<str:slug>/projects/<uuid:project_id>/issue-labels/<uuid:pk>/",
         LabelViewSet.as_view(
             {
                 "get": "retrieve",
@@ -74,28 +73,28 @@ urlpatterns = [
         name="project-issue-labels",
     ),
     path(
-        "workspaces/<str:slug>/projects/<str:project_id>/bulk-create-labels/",
+        "workspaces/<str:slug>/projects/<uuid:project_id>/bulk-create-labels/",
         BulkCreateIssueLabelsEndpoint.as_view(),
         name="project-bulk-labels",
     ),
     path(
-        "workspaces/<str:slug>/projects/<str:project_id>/bulk-delete-issues/",
+        "workspaces/<str:slug>/projects/<uuid:project_id>/bulk-delete-issues/",
         BulkDeleteIssuesEndpoint.as_view(),
         name="project-issues-bulk",
     ),
     path(
-        "workspaces/<str:slug>/projects/<str:project_id>/bulk-archive-issues/",
+        "workspaces/<str:slug>/projects/<uuid:project_id>/bulk-archive-issues/",
         BulkArchiveIssuesEndpoint.as_view(),
         name="bulk-archive-issues",
     ),
     ##
     path(
-        "workspaces/<str:slug>/projects/<str:project_id>/issues/<str:issue_id>/sub-issues/",
+        "workspaces/<str:slug>/projects/<uuid:project_id>/issues/<uuid:issue_id>/sub-issues/",
         SubIssuesEndpoint.as_view(),
         name="sub-issues",
     ),
     path(
-        "workspaces/<str:slug>/projects/<str:project_id>/issues/<str:issue_id>/issue-links/",
+        "workspaces/<str:slug>/projects/<uuid:project_id>/issues/<uuid:issue_id>/issue-links/",
         IssueLinkViewSet.as_view(
             {
                 "get": "list",
@@ -105,7 +104,7 @@ urlpatterns = [
         name="project-issue-links",
     ),
     path(
-        "workspaces/<str:slug>/projects/<str:project_id>/issues/<str:issue_id>/issue-links/<uuid:pk>/",
+        "workspaces/<str:slug>/projects/<uuid:project_id>/issues/<uuid:issue_id>/issue-links/<uuid:pk>/",
         IssueLinkViewSet.as_view(
             {
                 "get": "retrieve",
@@ -117,12 +116,12 @@ urlpatterns = [
         name="project-issue-links",
     ),
     path(
-        "workspaces/<str:slug>/projects/<str:project_id>/issues/<str:issue_id>/issue-attachments/",
+        "workspaces/<str:slug>/projects/<uuid:project_id>/issues/<uuid:issue_id>/issue-attachments/",
         IssueAttachmentEndpoint.as_view(),
         name="project-issue-attachments",
     ),
     path(
-        "workspaces/<str:slug>/projects/<str:project_id>/issues/<str:issue_id>/issue-attachments/<uuid:pk>/",
+        "workspaces/<str:slug>/projects/<uuid:project_id>/issues/<uuid:issue_id>/issue-attachments/<uuid:pk>/",
         IssueAttachmentEndpoint.as_view(),
         name="project-issue-attachments",
     ),
@@ -134,14 +133,14 @@ urlpatterns = [
     ## End Issues
     ## Issue Activity
     path(
-        "workspaces/<str:slug>/projects/<str:project_id>/issues/<str:issue_id>/history/",
+        "workspaces/<str:slug>/projects/<uuid:project_id>/issues/<uuid:issue_id>/history/",
         IssueActivityEndpoint.as_view(),
         name="project-issue-history",
     ),
     ## Issue Activity
     ## IssueComments
     path(
-        "workspaces/<str:slug>/projects/<str:project_id>/issues/<str:issue_id>/comments/",
+        "workspaces/<str:slug>/projects/<uuid:project_id>/issues/<uuid:issue_id>/comments/",
         IssueCommentViewSet.as_view(
             {
                 "get": "list",
@@ -151,7 +150,7 @@ urlpatterns = [
         name="project-issue-comment",
     ),
     path(
-        "workspaces/<str:slug>/projects/<str:project_id>/issues/<str:issue_id>/comments/<uuid:pk>/",
+        "workspaces/<str:slug>/projects/<uuid:project_id>/issues/<uuid:issue_id>/comments/<uuid:pk>/",
         IssueCommentViewSet.as_view(
             {
                 "get": "retrieve",
@@ -165,7 +164,7 @@ urlpatterns = [
     ## End IssueComments
     # Issue Subscribers
     path(
-        "workspaces/<str:slug>/projects/<str:project_id>/issues/<str:issue_id>/issue-subscribers/",
+        "workspaces/<str:slug>/projects/<uuid:project_id>/issues/<uuid:issue_id>/issue-subscribers/",
         IssueSubscriberViewSet.as_view(
             {
                 "get": "list",
@@ -175,12 +174,12 @@ urlpatterns = [
         name="project-issue-subscribers",
     ),
     path(
-        "workspaces/<str:slug>/projects/<str:project_id>/issues/<str:issue_id>/issue-subscribers/<uuid:subscriber_id>/",
+        "workspaces/<str:slug>/projects/<uuid:project_id>/issues/<uuid:issue_id>/issue-subscribers/<uuid:subscriber_id>/",
         IssueSubscriberViewSet.as_view({"delete": "destroy"}),
         name="project-issue-subscribers",
     ),
     path(
-        "workspaces/<str:slug>/projects/<str:project_id>/issues/<str:issue_id>/subscribe/",
+        "workspaces/<str:slug>/projects/<uuid:project_id>/issues/<uuid:issue_id>/subscribe/",
         IssueSubscriberViewSet.as_view(
             {
                 "get": "subscription_status",
@@ -193,7 +192,7 @@ urlpatterns = [
     ## End Issue Subscribers
     # Issue Reactions
     path(
-        "workspaces/<str:slug>/projects/<str:project_id>/issues/<str:issue_id>/reactions/",
+        "workspaces/<str:slug>/projects/<uuid:project_id>/issues/<uuid:issue_id>/reactions/",
         IssueReactionViewSet.as_view(
             {
                 "get": "list",
@@ -203,7 +202,7 @@ urlpatterns = [
         name="project-issue-reactions",
     ),
     path(
-        "workspaces/<str:slug>/projects/<str:project_id>/issues/<str:issue_id>/reactions/<str:reaction_code>/",
+        "workspaces/<str:slug>/projects/<uuid:project_id>/issues/<uuid:issue_id>/reactions/<str:reaction_code>/",
         IssueReactionViewSet.as_view(
             {
                 "delete": "destroy",
@@ -214,7 +213,7 @@ urlpatterns = [
     ## End Issue Reactions
     # Comment Reactions
     path(
-        "workspaces/<str:slug>/projects/<str:project_id>/comments/<uuid:comment_id>/reactions/",
+        "workspaces/<str:slug>/projects/<uuid:project_id>/comments/<uuid:comment_id>/reactions/",
         CommentReactionViewSet.as_view(
             {
                 "get": "list",
@@ -224,7 +223,7 @@ urlpatterns = [
         name="project-issue-comment-reactions",
     ),
     path(
-        "workspaces/<str:slug>/projects/<str:project_id>/comments/<uuid:comment_id>/reactions/<str:reaction_code>/",
+        "workspaces/<str:slug>/projects/<uuid:project_id>/comments/<uuid:comment_id>/reactions/<str:reaction_code>/",
         CommentReactionViewSet.as_view(
             {
                 "delete": "destroy",
@@ -235,14 +234,14 @@ urlpatterns = [
     ## End Comment Reactions
     ## IssueProperty
     path(
-        "workspaces/<str:slug>/projects/<str:project_id>/user-properties/",
+        "workspaces/<str:slug>/projects/<uuid:project_id>/user-properties/",
         IssueUserDisplayPropertyEndpoint.as_view(),
         name="project-issue-display-properties",
     ),
     ## IssueProperty End
     ## Issue Archives
     path(
-        "workspaces/<str:slug>/projects/<str:project_id>/archived-issues/",
+        "workspaces/<str:slug>/projects/<uuid:project_id>/archived-issues/",
         IssueArchiveViewSet.as_view(
             {
                 "get": "list",
@@ -251,7 +250,7 @@ urlpatterns = [
         name="project-issue-archive",
     ),
     path(
-        "workspaces/<str:slug>/projects/<str:project_id>/issues/<uuid:pk>/archive/",
+        "workspaces/<str:slug>/projects/<uuid:project_id>/issues/<uuid:pk>/archive/",
         IssueArchiveViewSet.as_view(
             {
                 "get": "retrieve",
@@ -264,7 +263,7 @@ urlpatterns = [
     ## End Issue Archives
     ## Issue Relation
     path(
-        "workspaces/<str:slug>/projects/<str:project_id>/issues/<str:issue_id>/issue-relation/",
+        "workspaces/<str:slug>/projects/<uuid:project_id>/issues/<uuid:issue_id>/issue-relation/",
         IssueRelationViewSet.as_view(
             {
                 "get": "list",
@@ -274,7 +273,7 @@ urlpatterns = [
         name="issue-relation",
     ),
     path(
-        "workspaces/<str:slug>/projects/<str:project_id>/issues/<str:issue_id>/remove-relation/",
+        "workspaces/<str:slug>/projects/<uuid:project_id>/issues/<uuid:issue_id>/remove-relation/",
         IssueRelationViewSet.as_view(
             {
                 "post": "remove_relation",
@@ -285,7 +284,7 @@ urlpatterns = [
     ## End Issue Relation
     ## Issue Drafts
     path(
-        "workspaces/<str:slug>/projects/<str:project_id>/issue-drafts/",
+        "workspaces/<str:slug>/projects/<uuid:project_id>/issue-drafts/",
         IssueDraftViewSet.as_view(
             {
                 "get": "list",
@@ -295,7 +294,7 @@ urlpatterns = [
         name="project-issue-draft",
     ),
     path(
-        "workspaces/<str:slug>/projects/<str:project_id>/issue-drafts/<uuid:pk>/",
+        "workspaces/<str:slug>/projects/<uuid:project_id>/issue-drafts/<uuid:pk>/",
         IssueDraftViewSet.as_view(
             {
                 "get": "retrieve",
@@ -305,9 +304,4 @@ urlpatterns = [
         ),
         name="project-issue-draft",
     ),
-    path(
-        "workspaces/<str:slug>/projects/<str:project_id>/bulk-operation-issues/",
-        BulkIssueOperationsEndpoint.as_view(),
-        name="bulk-operations-issues",
-    ),
 ]

apiserver/plane/app/urls/module.py

@@ -13,7 +13,7 @@ from plane.app.views import (
 
 urlpatterns = [
     path(
-        "workspaces/<str:slug>/projects/<str:project_id>/modules/",
+        "workspaces/<str:slug>/projects/<uuid:project_id>/modules/",
         ModuleViewSet.as_view(
             {
                 "get": "list",
@@ -23,7 +23,7 @@ urlpatterns = [
         name="project-modules",
     ),
     path(
-        "workspaces/<str:slug>/projects/<str:project_id>/modules/<uuid:pk>/",
+        "workspaces/<str:slug>/projects/<uuid:project_id>/modules/<uuid:pk>/",
         ModuleViewSet.as_view(
             {
                 "get": "retrieve",
@@ -35,7 +35,7 @@ urlpatterns = [
         name="project-modules",
     ),
     path(
-        "workspaces/<str:slug>/projects/<str:project_id>/issues/<uuid:issue_id>/modules/",
+        "workspaces/<str:slug>/projects/<uuid:project_id>/issues/<uuid:issue_id>/modules/",
         ModuleIssueViewSet.as_view(
             {
                 "post": "create_issue_modules",
@@ -44,7 +44,7 @@ urlpatterns = [
         name="issue-module",
     ),
     path(
-        "workspaces/<str:slug>/projects/<str:project_id>/modules/<uuid:module_id>/issues/",
+        "workspaces/<str:slug>/projects/<uuid:project_id>/modules/<uuid:module_id>/issues/",
         ModuleIssueViewSet.as_view(
             {
                 "post": "create_module_issues",
@@ -54,7 +54,7 @@ urlpatterns = [
         name="project-module-issues",
     ),
     path(
-        "workspaces/<str:slug>/projects/<str:project_id>/modules/<uuid:module_id>/issues/<uuid:issue_id>/",
+        "workspaces/<str:slug>/projects/<uuid:project_id>/modules/<uuid:module_id>/issues/<uuid:issue_id>/",
         ModuleIssueViewSet.as_view(
             {
                 "get": "retrieve",
@@ -66,7 +66,7 @@ urlpatterns = [
         name="project-module-issues",
     ),
     path(
-        "workspaces/<str:slug>/projects/<str:project_id>/modules/<uuid:module_id>/module-links/",
+        "workspaces/<str:slug>/projects/<uuid:project_id>/modules/<uuid:module_id>/module-links/",
         ModuleLinkViewSet.as_view(
             {
                 "get": "list",
@@ -76,7 +76,7 @@ urlpatterns = [
         name="project-issue-module-links",
     ),
     path(
-        "workspaces/<str:slug>/projects/<str:project_id>/modules/<uuid:module_id>/module-links/<uuid:pk>/",
+        "workspaces/<str:slug>/projects/<uuid:project_id>/modules/<uuid:module_id>/module-links/<uuid:pk>/",
         ModuleLinkViewSet.as_view(
             {
                 "get": "retrieve",
@@ -88,7 +88,7 @@ urlpatterns = [
         name="project-issue-module-links",
     ),
     path(
-        "workspaces/<str:slug>/projects/<str:project_id>/user-favorite-modules/",
+        "workspaces/<str:slug>/projects/<uuid:project_id>/user-favorite-modules/",
         ModuleFavoriteViewSet.as_view(
             {
                 "get": "list",
@@ -98,7 +98,7 @@ urlpatterns = [
         name="user-favorite-module",
     ),
     path(
-        "workspaces/<str:slug>/projects/<str:project_id>/user-favorite-modules/<uuid:module_id>/",
+        "workspaces/<str:slug>/projects/<uuid:project_id>/user-favorite-modules/<uuid:module_id>/",
         ModuleFavoriteViewSet.as_view(
             {
                 "delete": "destroy",
@@ -107,22 +107,22 @@ urlpatterns = [
         name="user-favorite-module",
     ),
     path(
-        "workspaces/<str:slug>/projects/<str:project_id>/modules/<uuid:module_id>/user-properties/",
+        "workspaces/<str:slug>/projects/<uuid:project_id>/modules/<uuid:module_id>/user-properties/",
         ModuleUserPropertiesEndpoint.as_view(),
         name="cycle-user-filters",
     ),
     path(
-        "workspaces/<str:slug>/projects/<str:project_id>/modules/<uuid:module_id>/archive/",
+        "workspaces/<str:slug>/projects/<uuid:project_id>/modules/<uuid:module_id>/archive/",
         ModuleArchiveUnarchiveEndpoint.as_view(),
         name="module-archive-unarchive",
     ),
     path(
-        "workspaces/<str:slug>/projects/<str:project_id>/archived-modules/",
+        "workspaces/<str:slug>/projects/<uuid:project_id>/archived-modules/",
         ModuleArchiveUnarchiveEndpoint.as_view(),
         name="module-archive-unarchive",
     ),
     path(
-        "workspaces/<str:slug>/projects/<str:project_id>/archived-modules/<uuid:pk>/",
+        "workspaces/<str:slug>/projects/<uuid:project_id>/archived-modules/<uuid:pk>/",
         ModuleArchiveUnarchiveEndpoint.as_view(),
         name="module-archive-unarchive",
     ),

apiserver/plane/app/urls/page.py

@@ -12,7 +12,7 @@ from plane.app.views import (
 
 urlpatterns = [
     path(
-        "workspaces/<str:slug>/projects/<str:project_id>/pages/",
+        "workspaces/<str:slug>/projects/<uuid:project_id>/pages/",
         PageViewSet.as_view(
             {
                 "get": "list",
@@ -22,7 +22,7 @@ urlpatterns = [
         name="project-pages",
     ),
     path(
-        "workspaces/<str:slug>/projects/<str:project_id>/pages/<uuid:pk>/",
+        "workspaces/<str:slug>/projects/<uuid:project_id>/pages/<uuid:pk>/",
         PageViewSet.as_view(
             {
                 "get": "retrieve",
@@ -34,7 +34,7 @@ urlpatterns = [
     ),
     # favorite pages
     path(
-        "workspaces/<str:slug>/projects/<str:project_id>/favorite-pages/<uuid:pk>/",
+        "workspaces/<str:slug>/projects/<uuid:project_id>/favorite-pages/<uuid:pk>/",
         PageFavoriteViewSet.as_view(
             {
                 "post": "create",
@@ -45,7 +45,7 @@ urlpatterns = [
     ),
     # archived pages
     path(
-        "workspaces/<str:slug>/projects/<str:project_id>/pages/<uuid:pk>/archive/",
+        "workspaces/<str:slug>/projects/<uuid:project_id>/pages/<uuid:pk>/archive/",
         PageViewSet.as_view(
             {
                 "post": "archive",
@@ -56,7 +56,7 @@ urlpatterns = [
     ),
     # lock and unlock
     path(
-        "workspaces/<str:slug>/projects/<str:project_id>/pages/<uuid:pk>/lock/",
+        "workspaces/<str:slug>/projects/<uuid:project_id>/pages/<uuid:pk>/lock/",
         PageViewSet.as_view(
             {
                 "post": "lock",
@@ -66,22 +66,22 @@ urlpatterns = [
         name="project-pages-lock-unlock",
     ),
     path(
-        "workspaces/<str:slug>/projects/<str:project_id>/pages/<uuid:pk>/transactions/",
+        "workspaces/<str:slug>/projects/<uuid:project_id>/pages/<uuid:pk>/transactions/",
         PageLogEndpoint.as_view(),
         name="page-transactions",
     ),
     path(
-        "workspaces/<str:slug>/projects/<str:project_id>/pages/<uuid:pk>/transactions/<uuid:transaction>/",
+        "workspaces/<str:slug>/projects/<uuid:project_id>/pages/<uuid:pk>/transactions/<uuid:transaction>/",
         PageLogEndpoint.as_view(),
         name="page-transactions",
     ),
     path(
-        "workspaces/<str:slug>/projects/<str:project_id>/pages/<uuid:pk>/sub-pages/",
+        "workspaces/<str:slug>/projects/<uuid:project_id>/pages/<uuid:pk>/sub-pages/",
         SubPagesEndpoint.as_view(),
         name="sub-page",
     ),
     path(
-        "workspaces/<str:slug>/projects/<str:project_id>/pages/<uuid:pk>/description/",
+        "workspaces/<str:slug>/projects/<uuid:project_id>/pages/<uuid:pk>/description/",
         PagesDescriptionViewSet.as_view(
             {
                 "get": "retrieve",

apiserver/plane/app/urls/project.py

@@ -30,7 +30,7 @@ urlpatterns = [
         name="project",
     ),
     path(
-        "workspaces/<str:slug>/projects/<str:pk>/",
+        "workspaces/<str:slug>/projects/<uuid:pk>/",
         ProjectViewSet.as_view(
             {
                 "get": "retrieve",
@@ -47,7 +47,7 @@ urlpatterns = [
         name="project-identifiers",
     ),
     path(
-        "workspaces/<str:slug>/projects/<str:project_id>/invitations/",
+        "workspaces/<str:slug>/projects/<uuid:project_id>/invitations/",
         ProjectInvitationsViewset.as_view(
             {
                 "get": "list",
@@ -57,7 +57,7 @@ urlpatterns = [
         name="project-member-invite",
     ),
     path(
-        "workspaces/<str:slug>/projects/<str:project_id>/invitations/<uuid:pk>/",
+        "workspaces/<str:slug>/projects/<uuid:project_id>/invitations/<uuid:pk>/",
         ProjectInvitationsViewset.as_view(
             {
                 "get": "retrieve",
@@ -82,12 +82,12 @@ urlpatterns = [
         name="user-project-roles",
     ),
     path(
-        "workspaces/<str:slug>/projects/<str:project_id>/join/<uuid:pk>/",
+        "workspaces/<str:slug>/projects/<uuid:project_id>/join/<uuid:pk>/",
         ProjectJoinEndpoint.as_view(),
         name="project-join",
     ),
     path(
-        "workspaces/<str:slug>/projects/<str:project_id>/members/",
+        "workspaces/<str:slug>/projects/<uuid:project_id>/members/",
         ProjectMemberViewSet.as_view(
             {
                 "get": "list",
@@ -97,7 +97,7 @@ urlpatterns = [
         name="project-member",
     ),
     path(
-        "workspaces/<str:slug>/projects/<str:project_id>/members/<uuid:pk>/",
+        "workspaces/<str:slug>/projects/<uuid:project_id>/members/<uuid:pk>/",
         ProjectMemberViewSet.as_view(
             {
                 "get": "retrieve",
@@ -108,7 +108,7 @@ urlpatterns = [
         name="project-member",
     ),
     path(
-        "workspaces/<str:slug>/projects/<str:project_id>/members/leave/",
+        "workspaces/<str:slug>/projects/<uuid:project_id>/members/leave/",
         ProjectMemberViewSet.as_view(
             {
                 "post": "leave",
@@ -117,17 +117,17 @@ urlpatterns = [
         name="project-member",
     ),
     path(
-        "workspaces/<str:slug>/projects/<str:project_id>/team-invite/",
+        "workspaces/<str:slug>/projects/<uuid:project_id>/team-invite/",
         AddTeamToProjectEndpoint.as_view(),
         name="projects",
     ),
     path(
-        "workspaces/<str:slug>/projects/<str:project_id>/project-views/",
+        "workspaces/<str:slug>/projects/<uuid:project_id>/project-views/",
         ProjectUserViewsEndpoint.as_view(),
         name="project-view",
     ),
     path(
-        "workspaces/<str:slug>/projects/<str:project_id>/project-members/me/",
+        "workspaces/<str:slug>/projects/<uuid:project_id>/project-members/me/",
         ProjectMemberUserEndpoint.as_view(),
         name="project-member-view",
     ),
@@ -142,7 +142,7 @@ urlpatterns = [
         name="project-favorite",
     ),
     path(
-        "workspaces/<str:slug>/user-favorite-projects/<str:project_id>/",
+        "workspaces/<str:slug>/user-favorite-projects/<uuid:project_id>/",
         ProjectFavoritesViewSet.as_view(
             {
                 "delete": "destroy",
@@ -156,7 +156,7 @@ urlpatterns = [
         name="project-covers",
     ),
     path(
-        "workspaces/<str:slug>/projects/<str:project_id>/project-deploy-boards/",
+        "workspaces/<str:slug>/projects/<uuid:project_id>/project-deploy-boards/",
         DeployBoardViewSet.as_view(
             {
                 "get": "list",
@@ -166,7 +166,7 @@ urlpatterns = [
         name="project-deploy-board",
     ),
     path(
-        "workspaces/<str:slug>/projects/<str:project_id>/project-deploy-boards/<uuid:pk>/",
+        "workspaces/<str:slug>/projects/<uuid:project_id>/project-deploy-boards/<uuid:pk>/",
         DeployBoardViewSet.as_view(
             {
                 "get": "retrieve",
@@ -177,7 +177,7 @@ urlpatterns = [
         name="project-deploy-board",
     ),
     path(
-        "workspaces/<str:slug>/projects/<str:project_id>/archive/",
+        "workspaces/<str:slug>/projects/<uuid:project_id>/archive/",
         ProjectArchiveUnarchiveEndpoint.as_view(),
         name="project-archive-unarchive",
     ),

apiserver/plane/app/urls/search.py

@@ -14,7 +14,7 @@ urlpatterns = [
         name="global-search",
     ),
     path(
-        "workspaces/<str:slug>/projects/<str:project_id>/search-issues/",
+        "workspaces/<str:slug>/projects/<uuid:project_id>/search-issues/",
         IssueSearchEndpoint.as_view(),
         name="project-issue-search",
     ),

apiserver/plane/app/urls/state.py

@@ -6,7 +6,7 @@ from plane.app.views import StateViewSet
 
 urlpatterns = [
     path(
-        "workspaces/<str:slug>/projects/<str:project_id>/states/",
+        "workspaces/<str:slug>/projects/<uuid:project_id>/states/",
         StateViewSet.as_view(
             {
                 "get": "list",
@@ -16,7 +16,7 @@ urlpatterns = [
         name="project-states",
     ),
     path(
-        "workspaces/<str:slug>/projects/<str:project_id>/states/<uuid:pk>/",
+        "workspaces/<str:slug>/projects/<uuid:project_id>/states/<uuid:pk>/",
         StateViewSet.as_view(
             {
                 "get": "retrieve",
@@ -27,7 +27,7 @@ urlpatterns = [
         name="project-state",
     ),
     path(
-        "workspaces/<str:slug>/projects/<str:project_id>/states/<uuid:pk>/mark-default/",
+        "workspaces/<str:slug>/projects/<uuid:project_id>/states/<uuid:pk>/mark-default/",
         StateViewSet.as_view(
             {
                 "post": "mark_as_default",

apiserver/plane/app/urls/views.py

@@ -11,7 +11,7 @@ from plane.app.views import (
 
 urlpatterns = [
     path(
-        "workspaces/<str:slug>/projects/<str:project_id>/views/",
+        "workspaces/<str:slug>/projects/<uuid:project_id>/views/",
         IssueViewViewSet.as_view(
             {
                 "get": "list",
@@ -21,7 +21,7 @@ urlpatterns = [
         name="project-view",
     ),
     path(
-        "workspaces/<str:slug>/projects/<str:project_id>/views/<uuid:pk>/",
+        "workspaces/<str:slug>/projects/<uuid:project_id>/views/<uuid:pk>/",
         IssueViewViewSet.as_view(
             {
                 "get": "retrieve",
@@ -64,7 +64,7 @@ urlpatterns = [
         name="global-view-issues",
     ),
     path(
-        "workspaces/<str:slug>/projects/<str:project_id>/user-favorite-views/",
+        "workspaces/<str:slug>/projects/<uuid:project_id>/user-favorite-views/",
         IssueViewFavoriteViewSet.as_view(
             {
                 "get": "list",
@@ -74,7 +74,7 @@ urlpatterns = [
         name="user-favorite-view",
     ),
     path(
-        "workspaces/<str:slug>/projects/<str:project_id>/user-favorite-views/<uuid:view_id>/",
+        "workspaces/<str:slug>/projects/<uuid:project_id>/user-favorite-views/<uuid:view_id>/",
         IssueViewFavoriteViewSet.as_view(
             {
                 "delete": "destroy",

apiserver/plane/app/views/__init__.py

@@ -151,9 +151,6 @@ from .issue.subscriber import (
     IssueSubscriberViewSet,
 )
 
-
-from .issue.bulk_operations import BulkIssueOperationsEndpoint
-
 from .module.base import (
     ModuleViewSet,
     ModuleLinkViewSet,

apiserver/plane/app/views/base.py

@@ -1,12 +1,12 @@
 # Python imports
 import traceback
-import uuid
-import zoneinfo
 
-# Django imports
+import zoneinfo
 from django.conf import settings
 from django.core.exceptions import ObjectDoesNotExist, ValidationError
 from django.db import IntegrityError
+
+# Django imports
 from django.urls import resolve
 from django.utils import timezone
 from django_filters.rest_framework import DjangoFilterBackend
@@ -24,8 +24,6 @@ from rest_framework.viewsets import ModelViewSet
 from plane.authentication.session import BaseSessionAuthentication
 from plane.utils.exception_logger import log_exception
 from plane.utils.paginator import BasePaginator
-from plane.db.models import ProjectIdentifier, IssueSequence
-from plane.utils.valid_int_uuid import is_uuid, is_int_or_uuid
 
 
 class TimezoneMixin:
@@ -62,44 +60,6 @@ class BaseViewSet(TimezoneMixin, ModelViewSet, BasePaginator):
 
     search_fields = []
 
-    def get_project_id(self, project_id):
-        # If the project_id is present and is not a UUID then get the project_id from the project_identifier
-        if project_id and not is_uuid(project_id):
-            project_identifier = (
-                ProjectIdentifier.objects.filter(
-                    workspace__slug=self.workspace_slug, name=project_id
-                )
-                .values("project_id")
-                .first()
-            )
-            # If the project_identifier is not present then raise ObjectDoesNotExist
-            if not project_identifier:
-                raise ObjectDoesNotExist
-
-            # Update the project_id in the kwargs
-            return project_identifier.get("project_id")
-        return project_id
-
-    def get_issue_id(self, issue_id, project_id):
-        # If the issue_id is present and is not a UUID then get the issue_id from the issue_identifier
-        if issue_id:
-            type, value = is_int_or_uuid(issue_id)
-            if type == "int":
-                issue_identifier = (
-                    IssueSequence.objects.filter(
-                        project_id=project_id, sequence=value
-                    )
-                    .values("issue_id")
-                    .first()
-                )
-            # If the issue_identifier is not present then raise ObjectDoesNotExist
-            if not issue_identifier:
-                raise ObjectDoesNotExist
-
-            # Update the issue_id in the kwargs
-            return issue_identifier.get("issue_id")
-        return issue_id
-
     def get_queryset(self):
         try:
             return self.model.objects.all()
@@ -156,22 +116,8 @@ class BaseViewSet(TimezoneMixin, ModelViewSet, BasePaginator):
 
     def dispatch(self, request, *args, **kwargs):
         try:
-            # Write identifier to the uuid
-            project_id = kwargs.get("project_id", None)
-            if project_id:
-                kwargs["project_id"] = self.get_project_id(project_id)
-
-            # Check issue_id is present in the kwargs
-            issue_id = kwargs.get("issue_id", None)
-            if issue_id:
-                kwargs["issue_id"] = self.get_issue_id(
-                    issue_id, kwargs["project_id"]
-                )
-
-            # If the project_id is not present then get the project_id from the project_identifier
             response = super().dispatch(request, *args, **kwargs)
 
-            # Print the number of queries if the debug is enabled
             if settings.DEBUG:
                 from django.db import connection
 
@@ -179,7 +125,6 @@ class BaseViewSet(TimezoneMixin, ModelViewSet, BasePaginator):
                     f"{request.method} - {request.get_full_path()} of Queries: {len(connection.queries)}"
                 )
 
-            # Return the response
             return response
         except Exception as exc:
             response = self.handle_exception(exc)
@@ -193,7 +138,7 @@ class BaseViewSet(TimezoneMixin, ModelViewSet, BasePaginator):
     def project_id(self):
         project_id = self.kwargs.get("project_id", None)
         if project_id:
-            return self.get_project_id(project_id)
+            return project_id
 
         if resolve(self.request.path_info).url_name == "project":
             return self.kwargs.get("pk", None)

apiserver/plane/app/views/issue/base.py

@@ -44,7 +44,6 @@ from plane.db.models import (
     IssueReaction,
     IssueSubscriber,
     Project,
-    IssueSequence,
 )
 from plane.utils.grouper import (
     issue_group_values,
@@ -59,7 +58,8 @@ from plane.utils.paginator import (
 )
 from .. import BaseAPIView, BaseViewSet
 from plane.utils.user_timezone_converter import user_timezone_converter
-from plane.utils.valid_int_uuid import is_int_or_uuid
+
+# Module imports
 
 
 class IssueListEndpoint(BaseAPIView):
@@ -197,23 +197,6 @@ class IssueViewSet(BaseViewSet):
         "workspace__id",
     ]
 
-    def process_issue_id(self, issue_id, project_id):
-        # Check if the pk is an integer or a UUID
-        type, value = is_int_or_uuid(issue_id)
-        if type == "int":
-            # Get the issue_id from the sequence_id
-            sequence = (
-                IssueSequence.objects.filter(
-                    sequence=value,
-                    project_id=project_id,
-                )
-                .values("issue_id")
-                .first()
-            )
-            # Call the parent dispatch method
-            return sequence.get("issue_id")
-        return issue_id
-
     def get_queryset(self):
         return (
             Issue.issue_objects.filter(
@@ -427,76 +410,67 @@ class IssueViewSet(BaseViewSet):
         return Response(serializer.errors, status=status.HTTP_400_BAD_REQUEST)
 
     def retrieve(self, request, slug, project_id, pk=None):
-        pk = self.process_issue_id(pk, project_id)
         issue = (
-            (
-                self.get_queryset()
-                .annotate(
-                    label_ids=Coalesce(
-                        ArrayAgg(
-                            "labels__id",
-                            distinct=True,
-                            filter=~Q(labels__id__isnull=True),
-                        ),
-                        Value([], output_field=ArrayField(UUIDField())),
+            self.get_queryset()
+            .filter(pk=pk)
+            .annotate(
+                label_ids=Coalesce(
+                    ArrayAgg(
+                        "labels__id",
+                        distinct=True,
+                        filter=~Q(labels__id__isnull=True),
                     ),
-                    assignee_ids=Coalesce(
-                        ArrayAgg(
-                            "assignees__id",
-                            distinct=True,
-                            filter=~Q(assignees__id__isnull=True)
-                            & Q(assignees__member_project__is_active=True),
-                        ),
-                        Value([], output_field=ArrayField(UUIDField())),
+                    Value([], output_field=ArrayField(UUIDField())),
+                ),
+                assignee_ids=Coalesce(
+                    ArrayAgg(
+                        "assignees__id",
+                        distinct=True,
+                        filter=~Q(assignees__id__isnull=True)
+                        & Q(assignees__member_project__is_active=True),
                     ),
-                    module_ids=Coalesce(
-                        ArrayAgg(
-                            "issue_module__module_id",
-                            distinct=True,
-                            filter=~Q(issue_module__module_id__isnull=True),
-                        ),
-                        Value([], output_field=ArrayField(UUIDField())),
+                    Value([], output_field=ArrayField(UUIDField())),
+                ),
+                module_ids=Coalesce(
+                    ArrayAgg(
+                        "issue_module__module_id",
+                        distinct=True,
+                        filter=~Q(issue_module__module_id__isnull=True),
+                    ),
+                    Value([], output_field=ArrayField(UUIDField())),
+                ),
+            )
+            .prefetch_related(
+                Prefetch(
+                    "issue_reactions",
+                    queryset=IssueReaction.objects.select_related(
+                        "issue", "actor"
                     ),
                 )
-                .prefetch_related(
-                    Prefetch(
-                        "issue_reactions",
-                        queryset=IssueReaction.objects.select_related(
-                            "issue", "actor"
-                        ),
-                    )
+            )
+            .prefetch_related(
+                Prefetch(
+                    "issue_attachment",
+                    queryset=IssueAttachment.objects.select_related("issue"),
                 )
-                .prefetch_related(
-                    Prefetch(
-                        "issue_attachment",
-                        queryset=IssueAttachment.objects.select_related(
-                            "issue"
-                        ),
-                    )
+            )
+            .prefetch_related(
+                Prefetch(
+                    "issue_link",
+                    queryset=IssueLink.objects.select_related("created_by"),
                 )
-                .prefetch_related(
-                    Prefetch(
-                        "issue_link",
-                        queryset=IssueLink.objects.select_related(
-                            "created_by"
-                        ),
-                    )
-                )
-                .annotate(
-                    is_subscribed=Exists(
-                        IssueSubscriber.objects.filter(
-                            workspace__slug=slug,
-                            project_id=project_id,
-                            issue_id=OuterRef("pk"),
-                            subscriber=request.user,
-                        )
+            )
+            .annotate(
+                is_subscribed=Exists(
+                    IssueSubscriber.objects.filter(
+                        workspace__slug=slug,
+                        project_id=project_id,
+                        issue_id=OuterRef("pk"),
+                        subscriber=request.user,
                     )
                 )
             )
-            .filter(pk=pk)
-            .first()
-        )
-
+        ).first()
         if not issue:
             return Response(
                 {"error": "The required object does not exist."},
@@ -507,7 +481,6 @@ class IssueViewSet(BaseViewSet):
         return Response(serializer.data, status=status.HTTP_200_OK)
 
     def partial_update(self, request, slug, project_id, pk=None):
-        pk = self.process_issue_id(pk, project_id)
         issue = self.get_queryset().filter(pk=pk).first()
 
         if not issue:
@@ -542,11 +515,8 @@ class IssueViewSet(BaseViewSet):
         return Response(serializer.errors, status=status.HTTP_400_BAD_REQUEST)
 
     def destroy(self, request, slug, project_id, pk=None):
-        pk = self.process_issue_id(pk, project_id)
         issue = Issue.objects.get(
-            workspace__slug=slug,
-            project_id=project_id,
-            pk=pk,
+            workspace__slug=slug, project_id=project_id, pk=pk
         )
         issue.delete()
         issue_activity.delay(

apiserver/plane/app/views/issue/bulk_operations.py

@@ -1,288 +0,0 @@
-# Python imports
-import json
-from datetime import datetime
-
-# Django imports
-from django.utils import timezone
-
-# Third Party imports
-from rest_framework.response import Response
-from rest_framework import status
-
-# Module imports
-from .. import BaseAPIView
-from plane.app.permissions import (
-    ProjectEntityPermission,
-)
-from plane.db.models import (
-    Project,
-    Issue,
-    IssueLabel,
-    IssueAssignee,
-)
-from plane.bgtasks.issue_activites_task import issue_activity
-
-
-class BulkIssueOperationsEndpoint(BaseAPIView):
-    permission_classes = [
-        ProjectEntityPermission,
-    ]
-
-    def post(self, request, slug, project_id):
-        issue_ids = request.data.get("issue_ids", [])
-        if not len(issue_ids):
-            return Response(
-                {"error": "Issue IDs are required"},
-                status=status.HTTP_400_BAD_REQUEST,
-            )
-
-        # Get all the issues
-        issues = (
-            Issue.objects.filter(
-                workspace__slug=slug, project_id=project_id, pk__in=issue_ids
-            )
-            .select_related("state")
-            .prefetch_related("labels", "assignees")
-        )
-        # Current epoch
-        epoch = int(timezone.now().timestamp())
-
-        # Project details
-        project = Project.objects.get(workspace__slug=slug, pk=project_id)
-        workspace_id = project.workspace_id
-
-        # Initialize arrays
-        bulk_update_issues = []
-        bulk_issue_activities = []
-        bulk_update_issue_labels = []
-        bulk_update_issue_assignees = []
-
-        properties = request.data.get("properties", {})
-
-        if properties.get("start_date", False) and properties.get("target_date", False):
-            if (
-                datetime.strptime(properties.get("start_date"), "%Y-%m-%d").date()
-                > datetime.strptime(properties.get("target_date"), "%Y-%m-%d").date()
-            ):
-                return Response(
-                    {
-                        "error_code": 4100,
-                        "error_message": "INVALID_ISSUE_DATES",
-                    },
-                    status=status.HTTP_400_BAD_REQUEST,
-                )
-
-        for issue in issues:
-
-            # Priority
-            if properties.get("priority", False):
-                bulk_issue_activities.append(
-                    {
-                        "type": "issue.activity.updated",
-                        "requested_data": json.dumps(
-                            {"priority": properties.get("priority")}
-                        ),
-                        "current_instance": json.dumps(
-                            {"priority": (issue.priority)}
-                        ),
-                        "issue_id": str(issue.id),
-                        "actor_id": str(request.user.id),
-                        "project_id": str(project_id),
-                        "epoch": epoch,
-                    }
-                )
-                issue.priority = properties.get("priority")
-
-            # State
-            if properties.get("state_id", False):
-                bulk_issue_activities.append(
-                    {
-                        "type": "issue.activity.updated",
-                        "requested_data": json.dumps(
-                            {"state": properties.get("state")}
-                        ),
-                        "current_instance": json.dumps(
-                            {"state": str(issue.state_id)}
-                        ),
-                        "issue_id": str(issue.id),
-                        "actor_id": str(request.user.id),
-                        "project_id": str(project_id),
-                        "epoch": epoch,
-                    }
-                )
-                issue.state_id = properties.get("state_id")
-
-            # Start date
-            if properties.get("start_date", False):
-                if (
-                    issue.target_date
-                    and not properties.get("target_date", False)
-                    and issue.target_date
-                    <= datetime.strptime(
-                        properties.get("start_date"), "%Y-%m-%d"
-                    ).date()
-                ):
-                    return Response(
-                        {
-                            "error_code": 4101,
-                            "error_message": "INVALID_ISSUE_START_DATE",
-                        },
-                        status=status.HTTP_400_BAD_REQUEST,
-                    )
-                bulk_issue_activities.append(
-                    {
-                        "type": "issue.activity.updated",
-                        "requested_data": json.dumps(
-                            {"start_date": properties.get("start_date")}
-                        ),
-                        "current_instance": json.dumps(
-                            {"start_date": str(issue.start_date)}
-                        ),
-                        "issue_id": str(issue.id),
-                        "actor_id": str(request.user.id),
-                        "project_id": str(project_id),
-                        "epoch": epoch,
-                    }
-                )
-                issue.start_date = properties.get("start_date")
-
-            # Target date
-            if properties.get("target_date", False):
-                if (
-                    issue.start_date
-                    and not properties.get("start_date", False)
-                    and issue.start_date
-                    >= datetime.strptime(
-                        properties.get("target_date"), "%Y-%m-%d"
-                    ).date()
-                ):
-                    return Response(
-                        {
-                            "error_code": 4102,
-                            "error_message": "INVALID_ISSUE_TARGET_DATE",
-                        },
-                        status=status.HTTP_400_BAD_REQUEST,
-                    )
-                bulk_issue_activities.append(
-                    {
-                        "type": "issue.activity.updated",
-                        "requested_data": json.dumps(
-                            {"target_date": properties.get("target_date")}
-                        ),
-                        "current_instance": json.dumps(
-                            {"target_date": str(issue.target_date)}
-                        ),
-                        "issue_id": str(issue.id),
-                        "actor_id": str(request.user.id),
-                        "project_id": str(project_id),
-                        "epoch": epoch,
-                    }
-                )
-                issue.target_date = properties.get("target_date")
-
-            bulk_update_issues.append(issue)
-
-            # Labels
-            if properties.get("label_ids", []):
-                for label_id in properties.get("label_ids", []):
-                    bulk_update_issue_labels.append(
-                        IssueLabel(
-                            issue=issue,
-                            label_id=label_id,
-                            created_by=request.user,
-                            project_id=project_id,
-                            workspace_id=workspace_id,
-                        )
-                    )
-                bulk_issue_activities.append(
-                    {
-                        "type": "issue.activity.updated",
-                        "requested_data": json.dumps(
-                            {"label_ids": properties.get("label_ids", [])}
-                        ),
-                        "current_instance": json.dumps(
-                            {
-                                "label_ids": [
-                                    str(label.id)
-                                    for label in issue.labels.all()
-                                ]
-                            }
-                        ),
-                        "issue_id": str(issue.id),
-                        "actor_id": str(request.user.id),
-                        "project_id": str(project_id),
-                        "epoch": epoch,
-                    }
-                )
-
-            # Assignees
-            if properties.get("assignee_ids", []):
-                for assignee_id in properties.get(
-                    "assignee_ids", issue.assignees
-                ):
-                    bulk_update_issue_assignees.append(
-                        IssueAssignee(
-                            issue=issue,
-                            assignee_id=assignee_id,
-                            created_by=request.user,
-                            project_id=project_id,
-                            workspace_id=workspace_id,
-                        )
-                    )
-                bulk_issue_activities.append(
-                    {
-                        "type": "issue.activity.updated",
-                        "requested_data": json.dumps(
-                            {
-                                "assignee_ids": properties.get(
-                                    "assignee_ids", []
-                                )
-                            }
-                        ),
-                        "current_instance": json.dumps(
-                            {
-                                "assignee_ids": [
-                                    str(assignee.id)
-                                    for assignee in issue.assignees.all()
-                                ]
-                            }
-                        ),
-                        "issue_id": str(issue.id),
-                        "actor_id": str(request.user.id),
-                        "project_id": str(project_id),
-                        "epoch": epoch,
-                    }
-                )
-
-        # Bulk update all the objects
-        Issue.objects.bulk_update(
-            bulk_update_issues,
-            [
-                "priority",
-                "start_date",
-                "target_date",
-                "state",
-            ],
-            batch_size=100,
-        )
-
-        # Create new labels
-        IssueLabel.objects.bulk_create(
-            bulk_update_issue_labels,
-            ignore_conflicts=True,
-            batch_size=100,
-        )
-
-        # Create new assignees
-        IssueAssignee.objects.bulk_create(
-            bulk_update_issue_assignees,
-            ignore_conflicts=True,
-            batch_size=100,
-        )
-        # update the issue activity
-        [
-            issue_activity.delay(**activity)
-            for activity in bulk_issue_activities
-        ]
-
-        return Response(status=status.HTTP_204_NO_CONTENT)

apiserver/plane/app/views/project/base.py

@@ -50,7 +50,6 @@ from plane.db.models import (
 )
 from plane.utils.cache import cache_response
 from plane.bgtasks.webhook_task import model_activity
-from plane.utils.valid_int_uuid import is_uuid
 
 
 class ProjectViewSet(BaseViewSet):
@@ -62,25 +61,6 @@ class ProjectViewSet(BaseViewSet):
         ProjectBasePermission,
     ]
 
-    def dispatch(self, request, *args, **kwargs):
-        pk = kwargs.get("pk")
-        # Check if the project identifier is a UUID
-        if pk and not is_uuid(pk):
-            # Check if the project identifier is a name
-            project = (
-                ProjectIdentifier.objects.filter(
-                    name=pk, workspace__slug=kwargs.get("slug")
-                )
-                .values("project")
-                .first()
-            )
-            if project:
-                # Update the project identifier with the UUID
-                kwargs["pk"] = project["project"]
-
-        # Call the parent dispatch method
-        return super().dispatch(request, *args, **kwargs)
-
     def get_queryset(self):
         sort_order = ProjectMember.objects.filter(
             member=self.request.user,
@@ -504,39 +484,32 @@ class ProjectIdentifierEndpoint(BaseAPIView):
     ]
 
     def get(self, request, slug):
-        # Get the identifier name
         name = request.GET.get("name", "").strip().upper()
 
-        # Check if the name is passed
         if name == "":
             return Response(
                 {"error": "Name is required"},
                 status=status.HTTP_400_BAD_REQUEST,
             )
 
-        # Check if the identifier exists
         exists = ProjectIdentifier.objects.filter(
             name=name, workspace__slug=slug
         ).values("id", "name", "project")
 
-        # Return the response
         return Response(
             {"exists": len(exists), "identifiers": exists},
             status=status.HTTP_200_OK,
         )
 
     def delete(self, request, slug):
-        # Get the identifier name
         name = request.data.get("name", "").strip().upper()
 
-        # Check if the name is passed
         if name == "":
             return Response(
                 {"error": "Name is required"},
                 status=status.HTTP_400_BAD_REQUEST,
             )
 
-        # Check if the identifier is used in any project
         if Project.objects.filter(
             identifier=name, workspace__slug=slug
         ).exists():
@@ -547,12 +520,10 @@ class ProjectIdentifierEndpoint(BaseAPIView):
                 status=status.HTTP_400_BAD_REQUEST,
             )
 
-        # Delete the identifier
         ProjectIdentifier.objects.filter(
             name=name, workspace__slug=slug
         ).delete()
 
-        # Return success
         return Response(
             status=status.HTTP_204_NO_CONTENT,
         )

apiserver/plane/bgtasks/webhook_task.py

@@ -442,53 +442,47 @@ def model_activity(
     slug,
     origin=None,
 ):
-    try:
-        """Function takes in two json and computes differences between keys of both the json"""
-        if current_instance is None:
-            webhook_activity.delay(
-                event=model_name,
-                verb="created",
-                field=None,
-                old_value=None,
-                new_value=None,
-                actor_id=actor_id,
-                slug=slug,
-                current_site=origin,
-                event_id=model_id,
-                old_identifier=None,
-                new_identifier=None,
-            )
-            return
-
-        # Load the current instance
-        current_instance = (
-            json.loads(current_instance)
-            if current_instance is not None
-            else None
+    """Function takes in two json and computes differences between keys of both the json"""
+    if current_instance is None:
+        webhook_activity.delay(
+            event=model_name,
+            verb="created",
+            field=None,
+            old_value=None,
+            new_value=None,
+            actor_id=actor_id,
+            slug=slug,
+            current_site=origin,
+            event_id=model_id,
+            old_identifier=None,
+            new_identifier=None,
         )
-
-        # Loop through all keys in requested data and check the current value and requested value
-        for key in requested_data:
-            # Check if key is present in current instance or not
-            if key in current_instance:
-                current_value = current_instance.get(key, None)
-                requested_value = requested_data.get(key, None)
-                if current_value != requested_value:
-                    webhook_activity.delay(
-                        event=model_name,
-                        verb="updated",
-                        field=key,
-                        old_value=current_value,
-                        new_value=requested_value,
-                        actor_id=actor_id,
-                        slug=slug,
-                        current_site=origin,
-                        event_id=model_id,
-                        old_identifier=None,
-                        new_identifier=None,
-                    )
-
-        return
-    except Exception as e:
-        log_exception(e)
         return
+
+    # Load the current instance
+    current_instance = (
+        json.loads(current_instance) if current_instance is not None else None
+    )
+
+    # Loop through all keys in requested data and check the current value and requested value
+    for key in requested_data:
+        # Check if key is present in current instance or not
+        if key in current_instance:
+            current_value = current_instance.get(key, None)
+            requested_value = requested_data.get(key, None)
+            if current_value != requested_value:
+                webhook_activity.delay(
+                    event=model_name,
+                    verb="updated",
+                    field=key,
+                    old_value=current_value,
+                    new_value=requested_value,
+                    actor_id=actor_id,
+                    slug=slug,
+                    current_site=origin,
+                    event_id=model_id,
+                    old_identifier=None,
+                    new_identifier=None,
+                )
+
+    return

apiserver/plane/db/migrations/0070_projectidentifier_current_active_and_more.py

@@ -1,58 +0,0 @@
-# Generated by Django 4.2.11 on 2024-06-25 07:32
-
-from django.db import migrations, models
-import django.db.models.deletion
-
-
-def migrate_current_active(apps, schema_editor):
-    ProjectIdentifier = apps.get_model("db", "ProjectIdentifier")
-
-    # Update current_active field with name field
-    ProjectIdentifier.objects.update(current_active=models.F("name"))
-
-
-class Migration(migrations.Migration):
-
-    dependencies = [
-        ("db", "0069_alter_account_provider_and_more"),
-    ]
-
-    operations = [
-        migrations.AddField(
-            model_name="projectidentifier",
-            name="current_active",
-            field=models.CharField(db_index=True, max_length=12, null=True),
-        ),
-        migrations.AlterField(
-            model_name="projectidentifier",
-            name="name",
-            field=models.CharField(db_index=True, max_length=12),
-        ),
-        migrations.AlterField(
-            model_name="projectidentifier",
-            name="project",
-            field=models.ForeignKey(
-                on_delete=django.db.models.deletion.CASCADE,
-                related_name="project_identifier",
-                to="db.project",
-            ),
-        ),
-        migrations.RunPython(migrate_current_active),
-        migrations.AlterField(
-            model_name="issue",
-            name="sequence_id",
-            field=models.IntegerField(
-                db_index=True, default=1, verbose_name="Issue Sequence ID"
-            ),
-        ),
-        migrations.AlterField(
-            model_name="issuesequence",
-            name="issue",
-            field=models.OneToOneField(
-                null=True,
-                on_delete=django.db.models.deletion.SET_NULL,
-                related_name="issue_sequence",
-                to="db.issue",
-            ),
-        ),
-    ]

apiserver/plane/db/models/issue.py

@@ -7,8 +7,9 @@ from django.contrib.postgres.fields import ArrayField
 from django.core.exceptions import ValidationError
 from django.core.validators import MaxValueValidator, MinValueValidator
 from django.db import models
+from django.db.models.signals import post_save
+from django.dispatch import receiver
 from django.utils import timezone
-from django.db import transaction
 
 # Module imports
 from plane.utils.html_processor import strip_tags
@@ -173,8 +174,6 @@ class Issue(ProjectBaseModel):
         ordering = ("-created_at",)
 
     def save(self, *args, **kwargs):
-        # is the model being saved for the first time?
-        is_new = self._state.adding
         # This means that the model isn't saved to the database yet
         if self.state is None:
             try:
@@ -206,43 +205,32 @@ class Issue(ProjectBaseModel):
                     self.completed_at = None
             except ImportError:
                 pass
-        # Atomic transaction to ensure that the sequence is created only if the issue is created
-        with transaction.atomic():
-            if is_new:
-                # Get the maximum display_id value from the database
-                last_id = IssueSequence.objects.filter(
-                    project=self.project
-                ).aggregate(largest=models.Max("sequence"))["largest"]
-                # aggregate can return None! Check it first.
-                # If it isn't none, just use the last ID specified (which should be the greatest) and add one to it
-                if last_id:
-                    self.sequence_id = last_id + 1
-                else:
-                    self.sequence_id = 1
 
-                largest_sort_order = Issue.objects.filter(
-                    project=self.project, state=self.state
-                ).aggregate(largest=models.Max("sort_order"))["largest"]
-                if largest_sort_order is not None:
-                    self.sort_order = largest_sort_order + 10000
+        if self._state.adding:
+            # Get the maximum display_id value from the database
+            last_id = IssueSequence.objects.filter(
+                project=self.project
+            ).aggregate(largest=models.Max("sequence"))["largest"]
+            # aggregate can return None! Check it first.
+            # If it isn't none, just use the last ID specified (which should be the greatest) and add one to it
+            if last_id:
+                self.sequence_id = last_id + 1
+            else:
+                self.sequence_id = 1
 
-            # Strip the html tags using html parser
-            self.description_stripped = (
-                None
-                if (
-                    self.description_html == ""
-                    or self.description_html is None
-                )
-                else strip_tags(self.description_html)
-            )
-            super(Issue, self).save(*args, **kwargs)
-            # Create a sequence for the issue
-            if is_new:
-                IssueSequence.objects.create(
-                    issue=self,
-                    sequence=self.sequence_id,
-                    project=self.project,
-                )
+            largest_sort_order = Issue.objects.filter(
+                project=self.project, state=self.state
+            ).aggregate(largest=models.Max("sort_order"))["largest"]
+            if largest_sort_order is not None:
+                self.sort_order = largest_sort_order + 10000
+
+        # Strip the html tags using html parser
+        self.description_stripped = (
+            None
+            if (self.description_html == "" or self.description_html is None)
+            else strip_tags(self.description_html)
+        )
+        super(Issue, self).save(*args, **kwargs)
 
     def __str__(self):
         """Return name of the issue"""
@@ -679,3 +667,14 @@ class IssueVote(ProjectBaseModel):
 
     def __str__(self):
         return f"{self.issue.name} {self.actor.email}"
+
+
+# TODO: Find a better method to save the model
+@receiver(post_save, sender=Issue)
+def create_issue_sequence(sender, instance, created, **kwargs):
+    if created:
+        IssueSequence.objects.create(
+            issue=instance,
+            sequence=instance.sequence_id,
+            project=instance.project,
+        )

apiserver/plane/db/models/project.py

@@ -206,6 +206,7 @@ class ProjectMember(ProjectBaseModel):
         return f"{self.member.email} <{self.project.name}>"
 
 
+# TODO: Remove workspace relation later
 class ProjectIdentifier(AuditModel):
     workspace = models.ForeignKey(
         "db.Workspace",
@@ -213,11 +214,10 @@ class ProjectIdentifier(AuditModel):
         related_name="project_identifiers",
         null=True,
     )
-    project = models.ForeignKey(
+    project = models.OneToOneField(
         Project, on_delete=models.CASCADE, related_name="project_identifier"
     )
-    name = models.CharField(max_length=12, db_index=True)
-    current_active = models.CharField(max_length=12, db_index=True, null=True)
+    name = models.CharField(max_length=12)
 
     class Meta:
         unique_together = ["name", "workspace"]

apiserver/plane/settings/local.py

@@ -8,10 +8,7 @@ DEBUG = True
 
 # Debug Toolbar settings
 INSTALLED_APPS += ("debug_toolbar",)  # noqa
-MIDDLEWARE += (  # noqa
-    "debug_toolbar.middleware.DebugToolbarMiddleware",
-    "kolo.middleware.KoloMiddleware",
-)
+MIDDLEWARE += ("debug_toolbar.middleware.DebugToolbarMiddleware",)  # noqa
 
 DEBUG_TOOLBAR_PATCH_SETTINGS = False
 

apiserver/plane/utils/valid_int_uuid.py

@@ -1,29 +0,0 @@
-# Python imports
-import uuid
-
-
-def is_uuid(value):
-    try:
-        # Check if the value is a valid UUID
-        uuid.UUID(str(value))
-        return True
-    except ValueError:
-        return False
-
-
-def is_int_or_uuid(value):
-    # Check if the value can be converted to an integer
-    try:
-        int_value = int(value)
-        return "int", int_value
-    except ValueError:
-        pass
-
-    # Check if the value can be converted to a UUID
-    try:
-        uuid_value = uuid.UUID(str(value))
-        return "uuid", uuid_value
-    except ValueError:
-        pass
-
-    return "", None