Skip to content

Commit 6b87e6f

Browse files
committed
Clean code after commons-ui update
Signed-off-by: Florent MILLOT <[email protected]>
1 parent 543a82a commit 6b87e6f

File tree

7 files changed

+8
-14
lines changed

7 files changed

+8
-14
lines changed

package.json

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -32,6 +32,7 @@
3232
"ag-grid-react": "^31.1.1",
3333
"core-js": "^3.6.4",
3434
"notistack": "^3.0.0",
35+
"oidc-client": "^1.10.1",
3536
"prop-types": "^15.7.2",
3637
"react": "^18.0.0",
3738
"react-dom": "^18.0.0",

src/components/App/app.test.tsx

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -14,7 +14,6 @@ import {
1414
ThemeProvider,
1515
} from '@mui/material/styles';
1616
import { SnackbarProvider } from '@gridsuite/commons-ui';
17-
import { UserManagerMock } from '@gridsuite/commons-ui/es/utils/UserManagerMock';
1817
import { CssBaseline } from '@mui/material';
1918
import { appRoutes } from '../../routes';
2019

src/module-commons-ui.d.ts

Lines changed: 0 additions & 3 deletions
This file was deleted.

src/redux/reducer.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -49,7 +49,7 @@ export type AppState = {
4949
[PARAM_LANGUAGE]: LanguageParameters;
5050

5151
userManager: UserManagerState;
52-
user: User | null; //TODO use true definition when commons-ui passed to typescript
52+
user: User | null; //TODO delete when migrated into commons-ui
5353
signInCallbackError: unknown;
5454
authenticationRouterError: unknown;
5555
showAuthenticationRouterLogin: boolean;

src/routes/router.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -79,7 +79,7 @@ export function appRoutes(): RouteObject[] {
7979
}
8080

8181
const AuthRouter: FunctionComponent<{
82-
userManager: (typeof AuthenticationRouter)['userManager'];
82+
userManager: Parameters<typeof AuthenticationRouter>[0]['userManager'];
8383
}> = (props, context) => {
8484
const signInCallbackError = useSelector(
8585
(state: AppState) => state.signInCallbackError

src/services/study.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,7 @@ import { APP_NAME } from '../utils/config-params';
1111

1212
const STUDY_URL = `${getRestBase()}/study/v1`;
1313

14-
//TODO delete when commons-ui will be in typescript
14+
//TODO delete when migrated into commons-ui
1515
export type ServerAbout = {
1616
type?: 'app' | 'server' | 'other';
1717
name?: string;

src/utils/language.ts

Lines changed: 4 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -7,11 +7,9 @@
77

88
import { LANG_ENGLISH, LANG_FRENCH, LANG_SYSTEM } from '@gridsuite/commons-ui';
99

10-
const supportedLanguages = [LANG_FRENCH, LANG_ENGLISH];
11-
//export type SupportedLanguagesType = typeof supportedLanguages[number]; //TODO when commons-ui in typescript
12-
export type SupportedLanguages = 'en' | 'fr';
13-
//export type LanguageParameters = SupportedLanguages | typeof LANG_SYSTEM; //TODO when commons-ui in typescript
14-
export type LanguageParameters = SupportedLanguages | 'sys';
10+
const supportedLanguages: string[] = [LANG_FRENCH, LANG_ENGLISH];
11+
export type SupportedLanguages = typeof supportedLanguages[number];
12+
export type LanguageParameters = SupportedLanguages | typeof LANG_SYSTEM;
1513

1614
export function getSystemLanguage(): SupportedLanguages {
1715
const systemLanguage = navigator.language.split(/[-_]/)[0];
@@ -25,6 +23,5 @@ export function getComputedLanguage(
2523
): SupportedLanguages {
2624
return language === LANG_SYSTEM
2725
? getSystemLanguage()
28-
: (language as SupportedLanguages);
29-
//TODO remove cast when commons-ui in typescript
26+
: language;
3027
}

0 commit comments

Comments
 (0)