Skip to content

Commit 9b844a2

Browse files
committed
clean types
Signed-off-by: David BRAQUART <[email protected]>
1 parent 0d1cd42 commit 9b844a2

File tree

6 files changed

+13
-30
lines changed

6 files changed

+13
-30
lines changed

src/pages/common/table-selection.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
/**
2-
* Copyright (c) 2024, RTE (http://www.rte-france.com)
2+
* Copyright (c) 2025, RTE (http://www.rte-france.com)
33
* This Source Code Form is subject to the terms of the Mozilla Public
44
* License, v. 2.0. If a copy of the MPL was not distributed with this
55
* file, You can obtain one at http://mozilla.org/MPL/2.0/.

src/pages/groups/modification/group-modification-dialog.tsx

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -15,7 +15,7 @@ import { yupResolver } from '@hookform/resolvers/yup';
1515
import { useForm } from 'react-hook-form';
1616
import { FunctionComponent, useCallback, useEffect, useMemo, useState } from 'react';
1717
import { CustomMuiDialog, FetchStatus, useSnackMessage } from '@gridsuite/commons-ui';
18-
import { GroupInfos, UpdateGroupInfos, UserAdminSrv, UserInfos } from '../../../services';
18+
import { GroupInfos, UserAdminSrv, UserInfos } from '../../../services';
1919

2020
interface GroupModificationDialogProps {
2121
groupInfos: GroupInfos | undefined;
@@ -86,7 +86,7 @@ const GroupModificationDialog: FunctionComponent<GroupModificationDialogProps> =
8686
const onSubmit = useCallback(
8787
(groupFormData: GroupModificationFormType) => {
8888
if (groupInfos?.id) {
89-
const newData: UpdateGroupInfos = {
89+
const newData: GroupInfos = {
9090
id: groupInfos.id,
9191
name: groupFormData.name,
9292
users: selectedUsers,

src/pages/groups/modification/group-modification-form.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,7 @@ import { TextInput } from '@gridsuite/commons-ui';
99
import Grid from '@mui/material/Grid';
1010
import React, { FunctionComponent } from 'react';
1111
import yup from '../../../utils/yup-config';
12-
import TableSelection from 'pages/common/table-selection';
12+
import TableSelection from '../../common/table-selection';
1313

1414
export const GROUP_NAME = 'name';
1515
export const SELECTED_USERS = 'users';

src/pages/profiles/profiles-table.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,7 @@ import {
1616
ITooltipParams,
1717
RowClickedEvent,
1818
SelectionChangedEvent,
19-
TextFilterParams
19+
TextFilterParams,
2020
} from 'ag-grid-community';
2121
import { useSnackMessage } from '@gridsuite/commons-ui';
2222
import DeleteConfirmationDialog from '../common/delete-confirmation-dialog';

src/pages/users/modification/user-modification-dialog.tsx

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,7 @@ import { yupResolver } from '@hookform/resolvers/yup';
1616
import { useForm } from 'react-hook-form';
1717
import { FunctionComponent, useCallback, useEffect, useMemo, useState } from 'react';
1818
import { CustomMuiDialog, FetchStatus, useSnackMessage } from '@gridsuite/commons-ui';
19-
import { GroupInfos, UpdateUserInfos, UserAdminSrv, UserInfos, UserProfile } from '../../../services';
19+
import { GroupInfos, UserAdminSrv, UserInfos, UserProfile } from '../../../services';
2020

2121
interface UserModificationDialogProps {
2222
userInfos: UserInfos | undefined;
@@ -109,13 +109,13 @@ const UserModificationDialog: FunctionComponent<UserModificationDialogProps> = (
109109
const onSubmit = useCallback(
110110
(userFormData: UserModificationFormType) => {
111111
if (userInfos) {
112-
const newData: UpdateUserInfos = {
112+
const newData: UserInfos = {
113113
sub: userInfos.sub, // can't be changed
114114
isAdmin: userInfos.isAdmin, // can't be changed
115115
profileName: userFormData.profileName ?? undefined,
116116
groups: selectedGroups,
117117
};
118-
UserAdminSrv.udpateUser(newData)
118+
UserAdminSrv.updateUser(newData)
119119
.catch((error) =>
120120
snackError({
121121
messageTxt: error.message,

src/services/user-admin.ts

Lines changed: 5 additions & 22 deletions
Original file line numberDiff line numberDiff line change
@@ -7,15 +7,11 @@
77

88
import { User } from 'oidc-client';
99
import { backendFetch, backendFetchJson, getRestBase } from '../utils/api-rest';
10-
import { extractUserSub, getToken, getUser } from '../utils/api';
10+
import { extractUserSub, getToken } from '../utils/api';
1111
import { UUID } from 'crypto';
1212

1313
const USER_ADMIN_URL = `${getRestBase()}/user-admin/v1`;
1414

15-
export function getUserSub(): Promise<unknown> {
16-
return extractUserSub(getUser());
17-
}
18-
1915
/*
2016
* fetchValidateUser is call from commons-ui AuthServices to validate user infos before setting state.user!
2117
*/
@@ -40,7 +36,7 @@ export function fetchValidateUser(user: User): Promise<boolean> {
4036

4137
export type UserInfos = {
4238
sub: string;
43-
profileName: string;
39+
profileName?: string;
4440
isAdmin: boolean;
4541
groups: string[];
4642
};
@@ -59,14 +55,7 @@ export function fetchUsers(): Promise<UserInfos[]> {
5955
}) as Promise<UserInfos[]>;
6056
}
6157

62-
export type UpdateUserInfos = {
63-
sub: string;
64-
profileName?: string;
65-
isAdmin?: boolean;
66-
groups: string[];
67-
};
68-
69-
export function udpateUser(userInfos: UpdateUserInfos) {
58+
export function updateUser(userInfos: UserInfos) {
7059
console.debug(`Updating a user...`);
7160

7261
return backendFetch(`${USER_ADMIN_URL}/users/${userInfos.sub}`, {
@@ -216,7 +205,7 @@ export function deleteProfiles(names: string[]): Promise<void> {
216205
}
217206

218207
export type GroupInfos = {
219-
id?: UUID;
208+
id: UUID;
220209
name: string;
221210
users: string[];
222211
};
@@ -235,13 +224,7 @@ export function fetchGroups(): Promise<GroupInfos[]> {
235224
}) as Promise<GroupInfos[]>;
236225
}
237226

238-
export type UpdateGroupInfos = {
239-
id: UUID;
240-
name: string;
241-
users: string[];
242-
};
243-
244-
export function udpateGroup(groupInfos: UpdateGroupInfos) {
227+
export function udpateGroup(groupInfos: GroupInfos) {
245228
console.debug(`Updating a group...`);
246229

247230
return backendFetch(`${USER_ADMIN_URL}/groups/${groupInfos.id}`, {

0 commit comments

Comments
 (0)