yujunjun/OpenMetadata
1
0
Fork 0
mirror of https://github.com/open-metadata/OpenMetadata.git synced 2026-01-07 13:07:22 +00:00
Code Issues Packages Projects Releases Wiki Activity

changed the after login API call method, to update the user details, from PUT to PATCH (#12287)

Browse Source
This commit is contained in:
Aniket Katkar 2023-07-05 14:47:18 +05:30 committed by GitHub
parent 9aede5e821
commit eaf4ab0541
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 8 additions and 29 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

37
openmetadata-ui/src/main/resources/ui/src/components/authentication/auth-provider/AuthProvider.tsx
View File

@ -19,6 +19,7 @@ import { LoginCallback } from '@okta/okta-react';
import appState from 'AppState';
import { AxiosError } from 'axios';
import { CookieStorage } from 'cookie-storage';
import { compare } from 'fast-json-patch';
import { AuthorizerConfiguration } from 'generated/configuration/authorizerConfiguration';
import { isEmpty, isNil, isNumber } from 'lodash';
import { observer } from 'mobx-react';
@ -36,7 +37,7 @@ import { useTranslation } from 'react-i18next';
import { useHistory, useLocation } from 'react-router-dom';
import axiosClient from 'rest/index';
import { fetchAuthenticationConfig, fetchAuthorizerConfig } from 'rest/miscAPI';
import { getLoggedInUser, updateUser } from 'rest/userAPI';
import { getLoggedInUser, updateUserDetail } from 'rest/userAPI';
import { NO_AUTH } from '../../../constants/auth.constants';
import { REDIRECT_PATHNAME, ROUTES } from '../../../constants/constants';
import { ClientErrors } from '../../../enums/axios.enum';
@ -211,35 +212,13 @@ export const AuthProvider = ({
const getUpdatedUser = (updatedData: User, existingData: User) => {
// PUT method for users api only excepts below fields
const {
isAdmin,
teams,
timezone,
name,
description,
displayName,
profile,
email,
isBot,
roles,
} = { ...existingData, ...updatedData };
const teamIds = teams?.map((team) => team.id);
const roleIds = roles?.map((role) => role.id);
updateUser({
isAdmin,
teams: teamIds,
timezone,
name,
description,
displayName,
profile,
email,
isBot,
roles: roleIds,
} as User)
const updatedUserData = { ...existingData, ...updatedData };
const jsonPatch = compare(existingData, updatedUserData);
updateUserDetail(existingData.id, jsonPatch)
.then((res) => {
if (res.data) {
appState.updateUserDetails(res.data);
if (res) {
appState.updateUserDetails({ ...existingData, ...res });
} else {
throw t('server.unexpected-response');
}