From 04ad748f155d903291923d1e5445fefd27332c46 Mon Sep 17 00:00:00 2001
From: Pranav C <pranavxc@gmail.com>
Date: Mon, 15 Nov 2021 13:28:11 +0530
Subject: [PATCH] fix: hide comments from viewer role

Signed-off-by: Pranav C <pranavxc@gmail.com>
---
 .../project/spreadsheet/components/expandedForm.vue         | 3 ++-
 packages/nc-gui/helpers/rolePermissionsEE.js                | 6 ++++--
 2 files changed, 6 insertions(+), 3 deletions(-)

diff --git a/packages/nc-gui/components/project/spreadsheet/components/expandedForm.vue b/packages/nc-gui/components/project/spreadsheet/components/expandedForm.vue
index 6996d259c7..9dfd3ce3f1 100644
--- a/packages/nc-gui/components/project/spreadsheet/components/expandedForm.vue
+++ b/packages/nc-gui/components/project/spreadsheet/components/expandedForm.vue
@@ -32,7 +32,7 @@
         </x-icon>
 
         <x-icon
-          v-if="!isNew"
+          v-if="!isNew && _isUIAllowed('rowComments')"
           icon-class="mr-2"
           tooltip="Toggle comments"
           small
@@ -246,6 +246,7 @@
     </div>
 
     <v-btn
+      v-if="_isUIAllowed('rowComments')"
       v-show="!toggleDrawer"
       class="comment-icon"
       color="primary"
diff --git a/packages/nc-gui/helpers/rolePermissionsEE.js b/packages/nc-gui/helpers/rolePermissionsEE.js
index 0f86083e65..9efe4658d6 100644
--- a/packages/nc-gui/helpers/rolePermissionsEE.js
+++ b/packages/nc-gui/helpers/rolePermissionsEE.js
@@ -7,11 +7,13 @@ export default {
     xcDatatableEditable: true,
     column: true,
     tableAttachment: true,
-    tableRowUpdate: true
+    tableRowUpdate: true,
+    rowComments: true
   },
   commenter: {
     airTable: true,
-    column: true
+    column: true,
+    rowComments: true
   },
   viewer: {
     airTable: true,