From c9d2ea36b8312d0c00d8a36b4cdb57fb4f41b0af Mon Sep 17 00:00:00 2001
From: Prateek Shourya <prateekshourya29@gmail.com>
Date: Tue, 30 Jan 2024 20:13:28 +0530
Subject: [PATCH] chore: allow guests/ viewers to comment. (#3515)

---
 .../issue-detail/inbox/main-content.tsx       |  2 +-
 .../comments/comment-create.tsx               |  5 ++--
 .../issue-detail/issue-activity/root.tsx      | 29 +++++++------------
 .../issues/issue-detail/main-content.tsx      |  2 +-
 web/components/issues/peek-overview/view.tsx  |  2 --
 5 files changed, 15 insertions(+), 25 deletions(-)

diff --git a/web/components/issues/issue-detail/inbox/main-content.tsx b/web/components/issues/issue-detail/inbox/main-content.tsx
index 2e612ad34..4a1f79bee 100644
--- a/web/components/issues/issue-detail/inbox/main-content.tsx
+++ b/web/components/issues/issue-detail/inbox/main-content.tsx
@@ -79,7 +79,7 @@ export const InboxIssueMainContent: React.FC<Props> = observer((props) => {
       </div>
 
       <div className="pb-12">
-        <IssueActivity workspaceSlug={workspaceSlug} projectId={projectId} issueId={issueId} disabled={!is_editable} />
+        <IssueActivity workspaceSlug={workspaceSlug} projectId={projectId} issueId={issueId} />
       </div>
     </>
   );
diff --git a/web/components/issues/issue-detail/issue-activity/comments/comment-create.tsx b/web/components/issues/issue-detail/issue-activity/comments/comment-create.tsx
index 28bd00b1f..172a0bb82 100644
--- a/web/components/issues/issue-detail/issue-activity/comments/comment-create.tsx
+++ b/web/components/issues/issue-detail/issue-activity/comments/comment-create.tsx
@@ -17,7 +17,6 @@ const fileService = new FileService();
 type TIssueCommentCreate = {
   workspaceSlug: string;
   activityOperations: TActivityOperations;
-  disabled: boolean;
   showAccessSpecifier?: boolean;
 };
 
@@ -40,7 +39,7 @@ const commentAccess: commentAccessType[] = [
 ];
 
 export const IssueCommentCreate: FC<TIssueCommentCreate> = (props) => {
-  const { workspaceSlug, activityOperations, disabled, showAccessSpecifier = false } = props;
+  const { workspaceSlug, activityOperations, showAccessSpecifier = false } = props;
   const workspaceStore = useWorkspace();
   const workspaceId = workspaceStore.getWorkspaceBySlug(workspaceSlug as string)?.id as string;
 
@@ -94,7 +93,7 @@ export const IssueCommentCreate: FC<TIssueCommentCreate> = (props) => {
                 }
                 submitButton={
                   <Button
-                    disabled={isSubmitting || disabled}
+                    disabled={isSubmitting}
                     variant="primary"
                     type="submit"
                     className="!px-2.5 !py-1.5 !text-xs"
diff --git a/web/components/issues/issue-detail/issue-activity/root.tsx b/web/components/issues/issue-detail/issue-activity/root.tsx
index 67d0a48f8..42d856b1e 100644
--- a/web/components/issues/issue-detail/issue-activity/root.tsx
+++ b/web/components/issues/issue-detail/issue-activity/root.tsx
@@ -13,7 +13,6 @@ type TIssueActivity = {
   workspaceSlug: string;
   projectId: string;
   issueId: string;
-  disabled: boolean;
 };
 
 type TActivityTabs = "all" | "activity" | "comments";
@@ -43,7 +42,7 @@ export type TActivityOperations = {
 };
 
 export const IssueActivity: FC<TIssueActivity> = observer((props) => {
-  const { workspaceSlug, projectId, issueId, disabled } = props;
+  const { workspaceSlug, projectId, issueId } = props;
   // hooks
   const { createComment, updateComment, removeComment } = useIssueDetail();
   const { setToastAlert } = useToast();
@@ -147,14 +146,11 @@ export const IssueActivity: FC<TIssueActivity> = observer((props) => {
                 activityOperations={activityOperations}
                 showAccessSpecifier={project.is_deployed}
               />
-              {!disabled && (
-                <IssueCommentCreate
-                  workspaceSlug={workspaceSlug}
-                  activityOperations={activityOperations}
-                  disabled={disabled}
-                  showAccessSpecifier={project.is_deployed}
-                />
-              )}
+              <IssueCommentCreate
+                workspaceSlug={workspaceSlug}
+                activityOperations={activityOperations}
+                showAccessSpecifier={project.is_deployed}
+              />
             </div>
           ) : activityTab === "activity" ? (
             <IssueActivityRoot issueId={issueId} />
@@ -166,14 +162,11 @@ export const IssueActivity: FC<TIssueActivity> = observer((props) => {
                 activityOperations={activityOperations}
                 showAccessSpecifier={project.is_deployed}
               />
-              {!disabled && (
-                <IssueCommentCreate
-                  workspaceSlug={workspaceSlug}
-                  activityOperations={activityOperations}
-                  disabled={disabled}
-                  showAccessSpecifier={project.is_deployed}
-                />
-              )}
+              <IssueCommentCreate
+                workspaceSlug={workspaceSlug}
+                activityOperations={activityOperations}
+                showAccessSpecifier={project.is_deployed}
+              />
             </div>
           )}
         </div>
diff --git a/web/components/issues/issue-detail/main-content.tsx b/web/components/issues/issue-detail/main-content.tsx
index 31400fde2..075525801 100644
--- a/web/components/issues/issue-detail/main-content.tsx
+++ b/web/components/issues/issue-detail/main-content.tsx
@@ -99,7 +99,7 @@ export const IssueMainContent: React.FC<Props> = observer((props) => {
         disabled={!is_editable}
       />
 
-      <IssueActivity workspaceSlug={workspaceSlug} projectId={projectId} issueId={issueId} disabled={!is_editable} />
+      <IssueActivity workspaceSlug={workspaceSlug} projectId={projectId} issueId={issueId} />
     </>
   );
 });
diff --git a/web/components/issues/peek-overview/view.tsx b/web/components/issues/peek-overview/view.tsx
index 5d2e486e6..82bda41d5 100644
--- a/web/components/issues/peek-overview/view.tsx
+++ b/web/components/issues/peek-overview/view.tsx
@@ -234,7 +234,6 @@ export const IssueView: FC<IIssueView> = observer((props) => {
                           workspaceSlug={workspaceSlug}
                           projectId={projectId}
                           issueId={issueId}
-                          disabled={disabled}
                         />
                       </div>
                     ) : (
@@ -255,7 +254,6 @@ export const IssueView: FC<IIssueView> = observer((props) => {
                               workspaceSlug={workspaceSlug}
                               projectId={projectId}
                               issueId={issueId}
-                              disabled={disabled}
                             />
                           </div>
                         </div>