fix: edit group (#3856)

Co-authored-by: kyle-ssg <[email protected]>
Flagsmith · Apr 30, 2024 · b25e6f8 · b25e6f8
1 parent bbf82f9
commit b25e6f8
Show file tree

Hide file tree

Showing 3 changed files with 18 additions and 8 deletions.
diff --git a/frontend/common/services/useGroup.ts b/frontend/common/services/useGroup.ts
@@ -11,7 +11,10 @@ export const groupService = service
   .injectEndpoints({
     endpoints: (builder) => ({
       createGroup: builder.mutation<Res['group'], Req['createGroup']>({
-        invalidatesTags: [{ id: 'LIST', type: 'Group' }],
+        invalidatesTags: [
+          { id: 'LIST', type: 'Group' },
+          { type: 'GroupSummary' },
+        ],
         queryFn: async (query, { dispatch }, _, baseQuery) => {
           //Create the group
           const { data, error } = await baseQuery({
@@ -47,7 +50,10 @@ export const groupService = service
         Res['groupAdmin'],
         Req['createGroupAdmin']
       >({
-        invalidatesTags: [{ id: 'LIST', type: 'Group' }],
+        invalidatesTags: [
+          { id: 'LIST', type: 'Group' },
+          { type: 'GroupSummary' },
+        ],
         query: (query: Req['createGroupAdmin']) => ({
           body: {},
           method: 'POST',
@@ -70,7 +76,10 @@ export const groupService = service
         Res['groupAdmin'],
         Req['deleteGroupAdmin']
       >({
-        invalidatesTags: [{ id: 'LIST', type: 'Group' }],
+        invalidatesTags: [
+          { id: 'LIST', type: 'Group' },
+          { type: 'GroupSummary' },
+        ],
         query: (query: Req['deleteGroupAdmin']) => ({
           body: {},
           method: 'POST',
@@ -93,6 +102,7 @@ export const groupService = service
         invalidatesTags: (res) => [
           { id: 'LIST', type: 'Group' },
           { id: res?.id, type: 'Group' },
+          { type: 'GroupSummary' },
         ],
         queryFn: async (query, { dispatch }, _, baseQuery) => {
           //Create the group

diff --git a/frontend/web/components/UserGroupList.tsx b/frontend/web/components/UserGroupList.tsx
@@ -110,7 +110,10 @@ const UserGroupsRow: FC<UserGroupsRowType> = ({
             <Button
               id='remove-group'
               type='button'
-              onClick={() => removeGroup(id, name)}
+              onClick={(e) => {
+                e.stopPropagation()
+                removeGroup(id, name)
+              }}
               className='btn btn-with-icon'
             >
               <Icon name='trash-2' width={20} fill='#656D7B' />

diff --git a/frontend/web/components/modals/CreateGroup.tsx b/frontend/web/components/modals/CreateGroup.tsx
@@ -448,9 +448,6 @@ const CreateGroup: FC<CreateGroupType> = ({ group, orgId, roles }) => {
       )}
     </div>
   )
-  if (error) {
-    return
-  }
   return isEdit ? (
     <Tabs uncontrolled tabClassName='px-0'>
       <TabItem
@@ -466,7 +463,7 @@ const CreateGroup: FC<CreateGroupType> = ({ group, orgId, roles }) => {
       <TabItem tabLabel={<div>Permissions</div>}>{editPermissionsEl}</TabItem>
     </Tabs>
   ) : (
-    editGroup
+    editGroupEl
   )
 }