Skip to content

Commit 12f80b9

Browse files
code rabbit suggestions implemented
1 parent d40a8da commit 12f80b9

File tree

5 files changed

+23
-7
lines changed

5 files changed

+23
-7
lines changed

admin-ui/app/locales/es/translation.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1885,7 +1885,7 @@
18851885
"agamaLabPolicyDesigner": "Diseñador de políticas de Agama Lab",
18861886
"gluuFlexAdminUiPolicyStoreDisplay": "Gluu Flex Admin UI Policy Store",
18871887
"auditPolicyStoreUrlUpdated": "Configuración de la URL del almacén de políticas actualizada",
1888-
"auditSyncRoleToScopesMappings": "Sincronizar función con asignaciones de ámbitos",
1888+
"auditSyncRoleToScopesMappings": "Sincronizar rol con asignaciones de ámbitos",
18891889
"auditSetPolicyStoreAsDefault": "Establecer almacén de políticas predeterminado"
18901890
},
18911891
"mappings": {

admin-ui/app/locales/pt/translation.json

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1734,13 +1734,13 @@
17341734
"point6": "6. Abra a tela de configuração do Repositório de Políticas Cedarling na Admin UI e adicione a URL do Repositório de Políticas copiada no campo Repositório de Políticas Remoto da Admin UI. Defina o campo Ponto de Recuperação de Políticas como Remoto para usar a URL do Repositório de Políticas remoto para o controle de acesso da Admin UI.",
17351735
"useRemotePolicyStore": "Recomenda-se definir como Padrão para a produção. Se definido como Padrão, será utilizado o armazenamento da interface administrativa para a autorização do Cedarling. Ative o modo Padrão e utilize o botão de atualização para armazenar ou atualizar as políticas do GitHub no servidor da interface administrativa.",
17361736
"updateRemotePolicyStoreOnServer": "Clique aqui para atualizar o JSON do repositório de políticas padrão com a versão disponível no URL remoto configurado.",
1737-
"policyUrlDisabledWhenDefault": "O URL da política está desativado quando Predefinido está selecionado.",
1737+
"policyUrlDisabledWhenDefault": "O URL da política está desativado quando Padrão está selecionado.",
17381738
"policyStoreUrlInvalidError": "O URL do repositório de políticas deve ser um URL HTTP ou HTTPS válido.",
17391739
"agamaLabPolicyDesigner": "Agama Lab's Policy Designer",
17401740
"gluuFlexAdminUiPolicyStoreDisplay": "Gluu Flex Admin UI Policy Store",
17411741
"auditPolicyStoreUrlUpdated": "Configuração do URL do repositório de políticas atualizada",
1742-
"auditSyncRoleToScopesMappings": "Sincronizar função com mapeamentos de âmbitos",
1743-
"auditSetPolicyStoreAsDefault": "Definir repositório de políticas predefinido"
1742+
"auditSyncRoleToScopesMappings": "Sincronizar função com mapeamentos de escopos",
1743+
"auditSetPolicyStoreAsDefault": "Definir repositório de políticas padrão"
17441744
},
17451745
"mappings": {
17461746
"note_prefix": "Configurar",

admin-ui/plugins/admin/components/Cedarling/CedarlingConfigPage.tsx

Lines changed: 10 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -210,7 +210,7 @@ const CedarlingConfigPage: React.FC = () => {
210210
resource: ADMIN_UI_CEDARLING_CONFIG,
211211
message: userMessage,
212212
client_id: client_id,
213-
payload: {},
213+
payload: { auiPolicyStoreUrl: auiConfig?.auiPolicyStoreUrl ?? auiPolicyStoreUrl ?? '' },
214214
})
215215
} catch (error) {
216216
console.error('Error updating Cedarling configuration:', error)
@@ -224,7 +224,15 @@ const CedarlingConfigPage: React.FC = () => {
224224
setIsLoading(false)
225225
}
226226
},
227-
[dispatch, setRemotePolicyStoreAsDefaultMutation, userinfo, client_id, t],
227+
[
228+
dispatch,
229+
setRemotePolicyStoreAsDefaultMutation,
230+
userinfo,
231+
client_id,
232+
t,
233+
auiConfig?.auiPolicyStoreUrl,
234+
auiPolicyStoreUrl,
235+
],
228236
)
229237

230238
const handleInputChange = useCallback((e: React.ChangeEvent<HTMLInputElement>) => {

admin-ui/plugins/admin/components/Mapping/RolePermissionMappingPage.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -75,7 +75,7 @@ const RolePermissionMappingPage: React.FC = React.memo(function RolePermissionMa
7575
<GluuViewWrapper canShow={canReadMapping}>
7676
<Box>
7777
{mapping.length === 0 ? (
78-
<Alert severity="info" className={classes.errorAlert}>
78+
<Alert severity="info" className={classes.infoEmptyState}>
7979
{t('messages.no_role_mappings_found')}
8080
</Alert>
8181
) : (

admin-ui/plugins/admin/components/Mapping/styles/MappingPage.style.ts

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -202,4 +202,12 @@ export const useStyles = makeStyles<MappingPageStyleParams>()((_theme, { isDark,
202202
errorAlert: {
203203
marginBottom: MAPPING_SPACING.CARD_MARGIN_BOTTOM,
204204
},
205+
206+
infoEmptyState: {
207+
marginBottom: MAPPING_SPACING.CARD_MARGIN_BOTTOM,
208+
backgroundColor: theme.infoAlert.background,
209+
border: `1px solid ${theme.infoAlert.border}`,
210+
borderRadius: MAPPING_SPACING.INFO_ALERT_BORDER_RADIUS,
211+
padding: `${MAPPING_SPACING.INFO_ALERT_PADDING_VERTICAL}px ${MAPPING_SPACING.INFO_ALERT_PADDING_HORIZONTAL}px`,
212+
},
205213
}))

0 commit comments

Comments
 (0)