diff --git a/openmetadata-ui/src/main/resources/ui/src/auth-provider/AuthProvider.tsx b/openmetadata-ui/src/main/resources/ui/src/auth-provider/AuthProvider.tsx index a1f31c4859d..27a9e4f5718 100644 --- a/openmetadata-ui/src/main/resources/ui/src/auth-provider/AuthProvider.tsx +++ b/openmetadata-ui/src/main/resources/ui/src/auth-provider/AuthProvider.tsx @@ -15,7 +15,6 @@ import { AxiosResponse } from 'axios'; import { CookieStorage } from 'cookie-storage'; import { isEmpty, isNil } from 'lodash'; import { observer } from 'mobx-react'; -import { NewUser } from 'Models'; import { UserManager, WebStorageStateStore } from 'oidc-client'; import React, { ComponentType, @@ -36,7 +35,6 @@ import axiosClient from '../axiosAPIs'; import { fetchAuthorizerConfig } from '../axiosAPIs/miscAPI'; import { getLoggedInUser, getUserByName } from '../axiosAPIs/userAPI'; import Loader from '../components/Loader/Loader'; -import { FirstTimeUserModal } from '../components/Modals/FirstTimeUserModal/FirstTimeUserModal'; import { COOKIE_VERSION } from '../components/Modals/WhatsNewModal/whatsNewData'; import { oidcTokenKey, ROUTES } from '../constants/constants'; import { ClientErrors } from '../enums/axios.enum'; @@ -72,13 +70,9 @@ const AuthProvider: FunctionComponent = ({ const location = useLocation(); const history = useHistory(); const showToast = useToastContext(); - const { - isAuthenticatedRoute, - isFirstTimeUser, - isSignedIn, - isSigningIn, - isSignedOut, - } = useAuth(location.pathname); + const { isFirstTimeUser, isSignedIn, isSigningIn, isSignedOut } = useAuth( + location.pathname + ); const oidcUserToken = cookieStorage.getItem(oidcTokenKey); const [loading, setLoading] = useState(true); @@ -187,14 +181,14 @@ const AuthProvider: FunctionComponent = ({ }); }; - const handleFirstTourModal = (skip: boolean) => { - appState.newUser = {} as NewUser; - if (skip) { - history.push(ROUTES.HOME); - } else { - // TODO: Route to tour page - } - }; + // const handleFirstTourModal = (skip: boolean) => { + // appState.newUser = {} as NewUser; + // if (skip) { + // history.push(ROUTES.HOME); + // } else { + // // TODO: Route to tour page + // } + // }; useEffect(() => { fetchAuthConfig(); @@ -276,12 +270,13 @@ const AuthProvider: FunctionComponent = ({ )} - {isAuthenticatedRoute && isFirstTimeUser ? ( + {/* TODO: Uncomment below lines to show Welcome modal on Sign-up */} + {/* {isAuthenticatedRoute && isFirstTimeUser ? ( handleFirstTourModal(true)} onSave={() => handleFirstTourModal(false)} /> - ) : null} + ) : null} */} ) : null} diff --git a/openmetadata-ui/src/main/resources/ui/src/components/app-bar/Appbar.tsx b/openmetadata-ui/src/main/resources/ui/src/components/app-bar/Appbar.tsx index 328a9586a1b..5bfa1bb0a8e 100644 --- a/openmetadata-ui/src/main/resources/ui/src/components/app-bar/Appbar.tsx +++ b/openmetadata-ui/src/main/resources/ui/src/components/app-bar/Appbar.tsx @@ -62,9 +62,7 @@ const Appbar: React.FC = (): JSX.Element => { const searchQuery = match?.params?.searchQuery; const [searchValue, setSearchValue] = useState(searchQuery); const [isOpen, setIsOpen] = useState(true); - const [isFeatureModalOpen, setIsFeatureModalOpen] = useState(() => { - return !isFirstTimeUser && cookieStorage.getItem(COOKIE_VERSION) !== 'true'; - }); + const [isFeatureModalOpen, setIsFeatureModalOpen] = useState(false); const [version, setVersion] = useState(''); @@ -143,7 +141,8 @@ const Appbar: React.FC = (): JSX.Element => { useEffect(() => { setIsFeatureModalOpen( - !isFirstTimeUser && cookieStorage.getItem(COOKIE_VERSION) !== 'true' + // TODO: Add !isFirstTimeUser to condition if showing Welcome Modal + cookieStorage.getItem(COOKIE_VERSION) !== 'true' ); }, [isFirstTimeUser]);