feat(admin-ui): unable to add description of an existing permission i…

…n config-api #66
GluuFederation · Feb 14, 2022 · 806f4fe · 806f4fe
1 parent 246a7c9
commit 806f4fe
Show file tree

Hide file tree

Showing 4 changed files with 15 additions and 5 deletions.
diff --git a/admin-ui/app/locales/en/translation.json b/admin-ui/app/locales/en/translation.json
@@ -373,6 +373,7 @@
     "client_encryption_signing": "Encryption/Signing",
     "client_scripts": "Client Scripts",
     "config_api_roles": "Config API Roles",
+    "config_api_permissions": "Config API Permissions",
     "connectionUri": "Connection URI",
     "custom_scripts": "All Custom Scripts",
     "database_status_title": "Database status",
@@ -392,6 +393,7 @@
     "public_keys": "Public Keys",
     "profile_detail": "Profile Details",
     "roles": "Config API Roles",
+    "permissions": "Config API Permissions",
     "scopes": "OpenID/OAuth Scopes",
     "scripts": "Custom Interception Scripts",
     "schemaName": "Schema Name"

diff --git a/admin-ui/plugins/admin/components/Permissions/UiPermListPage.js b/admin-ui/plugins/admin/components/Permissions/UiPermListPage.js
@@ -1,5 +1,5 @@
 import React, { useEffect, useState } from 'react'
-import MaterialTable from '@material-table/core';
+import MaterialTable from '@material-table/core'
 import { Paper } from '@material-ui/core'
 import UiPermDetailPage from './UiPermDetailPage'
 import { Badge } from 'reactstrap'
@@ -60,7 +60,7 @@ function UiPermListPage({ apiPerms, permissions, loading, dispatch }) {
   }
   return (
     <Card>
-      <GluuRibbon title={t('titles.roles')} fromLeft />
+      <GluuRibbon title={t('titles.permissions')} fromLeft />
       <CardBody>
         <FormGroup row />
         <FormGroup row />
@@ -73,7 +73,7 @@ function UiPermListPage({ apiPerms, permissions, loading, dispatch }) {
               {
                 title: `${t('fields.name')}`,
                 field: 'permission',
-                editable:false,
+                editable: false,
                 width: '50%',
                 render: (rowData) => (
                   <Badge color="info">{rowData.permission}</Badge>

diff --git a/admin-ui/plugins/admin/redux/reducers/ApiPermissionReducer.js b/admin-ui/plugins/admin/redux/reducers/ApiPermissionReducer.js
@@ -54,9 +54,14 @@ export default function apiPermissionReducer(state = INIT_STATE, action) {
       return handleLoading()
     case EDIT_PERMISSION_RESPONSE:
       if (action.payload.data) {
+        let currentItems = [...state.items]
+        currentItems.filter(
+          (item) => item.permission === action.payload.data.permission,
+        )
+        currentItems.push(action.payload.data)
         return {
           ...state,
-          items: [...state.items],
+          items: currentItems,
           loading: false,
         }
       } else {

diff --git a/admin-ui/plugins/admin/redux/reducers/ApiRoleReducer.js b/admin-ui/plugins/admin/redux/reducers/ApiRoleReducer.js
@@ -55,9 +55,12 @@ export default function apiRoleReducer(state = INIT_STATE, action) {
       return handleLoading()
     case EDIT_ROLE_RESPONSE:
       if (action.payload.data) {
+        let currentItems = [...state.items]
+        currentItems.filter((item) => item.role === action.payload.data.role)
+        currentItems.push(action.payload.data)
         return {
           ...state,
-          items: [...state.items],
+          items: currentItems,
           loading: false,
         }
       } else {