Commit b5f3bbea authored by JzoNg's avatar JzoNg

update cache in appNav after app info updated

parent a192ae93
...@@ -88,10 +88,19 @@ const AppDetailLayout: FC<IAppDetailLayoutProps> = (props) => { ...@@ -88,10 +88,19 @@ const AppDetailLayout: FC<IAppDetailLayoutProps> = (props) => {
useEffect(() => { useEffect(() => {
setAppDetail() setAppDetail()
fetchAppDetail({ url: '/apps', id: appId }).then((res) => { fetchAppDetail({ url: '/apps', id: appId }).then((res) => {
setAppDetail(res) // redirections
setNavigation(getNavigations(appId, isCurrentWorkspaceManager, res.mode)) if ((res.mode === 'workflow' || res.mode === 'advanced-chat') && (pathname).endsWith('configuration')) {
router.replace(`/app/${appId}/workflow`)
}
else if ((res.mode !== 'workflow' && res.mode !== 'advanced-chat') && (pathname).endsWith('workflow')) {
router.replace(`/app/${appId}/configuration`)
}
else {
setAppDetail(res)
setNavigation(getNavigations(appId, isCurrentWorkspaceManager, res.mode))
}
}) })
}, [appId, getNavigations, isCurrentWorkspaceManager, setAppDetail]) }, [appId, getNavigations, isCurrentWorkspaceManager, pathname, router, setAppDetail])
if (!appDetail) { if (!appDetail) {
return ( return (
...@@ -101,12 +110,6 @@ const AppDetailLayout: FC<IAppDetailLayoutProps> = (props) => { ...@@ -101,12 +110,6 @@ const AppDetailLayout: FC<IAppDetailLayoutProps> = (props) => {
) )
} }
// redirections
if ((appDetail.mode === 'workflow' || appDetail.mode === 'advanced-chat') && (pathname).endsWith('configuration'))
router.replace(`/app/${appId}/workflow`)
if ((appDetail.mode !== 'workflow' && appDetail.mode !== 'advanced-chat') && (pathname).endsWith('workflow'))
router.replace(`/app/${appId}/configuration`)
return ( return (
<div className={cn(s.app, 'flex', 'overflow-hidden')}> <div className={cn(s.app, 'flex', 'overflow-hidden')}>
{appDetail && ( {appDetail && (
......
'use client' 'use client'
import { useCallback, useState } from 'react' import { useCallback, useEffect, useState } from 'react'
import { useTranslation } from 'react-i18next' import { useTranslation } from 'react-i18next'
import { useParams, usePathname } from 'next/navigation' import { useParams } from 'next/navigation'
import useSWR from 'swr' // import useSWR from 'swr'
import useSWRInfinite from 'swr/infinite' import useSWRInfinite from 'swr/infinite'
import { flatten } from 'lodash-es' import { flatten } from 'lodash-es'
import produce from 'immer'
import Nav from '../nav' import Nav from '../nav'
import { type NavItem } from '../nav/nav-selector'
import { Robot, RobotActive } from '../../base/icons/src/public/header-nav/studio' import { Robot, RobotActive } from '../../base/icons/src/public/header-nav/studio'
import { fetchAppDetail, fetchAppList } from '@/service/apps' import { fetchAppList } from '@/service/apps'
import CreateAppDialog from '@/app/components/app/create-app-dialog' import CreateAppDialog from '@/app/components/app/create-app-dialog'
import type { AppListResponse } from '@/models/app' import type { AppListResponse } from '@/models/app'
import { useAppContext } from '@/context/app-context' import { useAppContext } from '@/context/app-context'
import { useStore as useAppStore } from '@/app/components/app/store'
const getKey = (pageIndex: number, previousPageData: AppListResponse) => { const getKey = (pageIndex: number, previousPageData: AppListResponse) => {
if (!pageIndex || previousPageData.has_more) if (!pageIndex || previousPageData.has_more)
...@@ -21,19 +24,58 @@ const getKey = (pageIndex: number, previousPageData: AppListResponse) => { ...@@ -21,19 +24,58 @@ const getKey = (pageIndex: number, previousPageData: AppListResponse) => {
const AppNav = () => { const AppNav = () => {
const { t } = useTranslation() const { t } = useTranslation()
const { appId } = useParams()
const { isCurrentWorkspaceManager } = useAppContext() const { isCurrentWorkspaceManager } = useAppContext()
const { appDetail } = useAppStore()
const [showNewAppDialog, setShowNewAppDialog] = useState(false) const [showNewAppDialog, setShowNewAppDialog] = useState(false)
const { appId } = useParams() const [navItems, setNavItems] = useState<NavItem[]>([])
const isAppDetailPage = usePathname().split('/').includes('app')
const { data: currentApp } = useSWR((appId && isAppDetailPage) ? { url: '/apps', id: appId } : null, fetchAppDetail)
const { data: appsData, setSize } = useSWRInfinite(appId ? getKey : () => null, fetchAppList, { revalidateFirstPage: false }) const { data: appsData, setSize } = useSWRInfinite(appId ? getKey : () => null, fetchAppList, { revalidateFirstPage: false })
const appItems = flatten(appsData?.map(appData => appData.data))
const handleLoadmore = useCallback(() => { const handleLoadmore = useCallback(() => {
setSize(size => size + 1) setSize(size => size + 1)
}, [setSize]) }, [setSize])
useEffect(() => {
if (appsData) {
const appItems = flatten(appsData?.map(appData => appData.data))
const navItems = appItems.map((app) => {
const link = ((isCurrentWorkspaceManager, app) => {
if (!isCurrentWorkspaceManager) {
return `/app/${app.id}/overview`
}
else {
if (app.mode === 'workflow' || app.mode === 'advanced-chat')
return `/app/${app.id}/workflow`
else
return `/app/${app.id}/configuration`
}
})(isCurrentWorkspaceManager, app)
return {
id: app.id,
icon: app.icon,
icon_background: app.icon_background,
name: app.name,
link,
}
})
setNavItems(navItems)
}
}, [appsData, isCurrentWorkspaceManager, setNavItems])
// update current app name
useEffect(() => {
if (appDetail) {
const newNavItems = produce(navItems, (draft: NavItem[]) => {
navItems.forEach((app, index) => {
if (app.id === appDetail.id)
draft[index].name = appDetail.name
})
})
setNavItems(newNavItems)
}
}, [appDetail, navItems])
return ( return (
<> <>
<Nav <Nav
...@@ -42,14 +84,8 @@ const AppNav = () => { ...@@ -42,14 +84,8 @@ const AppNav = () => {
text={t('common.menus.apps')} text={t('common.menus.apps')}
activeSegment={['apps', 'app']} activeSegment={['apps', 'app']}
link='/apps' link='/apps'
curNav={currentApp} curNav={appDetail}
navs={appItems.map(item => ({ navs={navItems}
id: item.id,
name: item.name,
link: `/app/${item.id}/${isCurrentWorkspaceManager ? 'configuration' : 'overview'}`,
icon: item.icon,
icon_background: item.icon_background,
}))}
createText={t('common.menus.newApp')} createText={t('common.menus.newApp')}
onCreate={() => setShowNewAppDialog(true)} onCreate={() => setShowNewAppDialog(true)}
onLoadmore={handleLoadmore} onLoadmore={handleLoadmore}
......
...@@ -7,6 +7,7 @@ import classNames from 'classnames' ...@@ -7,6 +7,7 @@ import classNames from 'classnames'
import type { INavSelectorProps } from './nav-selector' import type { INavSelectorProps } from './nav-selector'
import NavSelector from './nav-selector' import NavSelector from './nav-selector'
import { ArrowNarrowLeft } from '@/app/components/base/icons/src/vender/line/arrows' import { ArrowNarrowLeft } from '@/app/components/base/icons/src/vender/line/arrows'
import { useStore as useAppStore } from '@/app/components/app/store'
type INavProps = { type INavProps = {
icon: React.ReactNode icon: React.ReactNode
...@@ -28,6 +29,7 @@ const Nav = ({ ...@@ -28,6 +29,7 @@ const Nav = ({
onCreate, onCreate,
onLoadmore, onLoadmore,
}: INavProps) => { }: INavProps) => {
const { setAppDetail } = useAppStore()
const [hovered, setHovered] = useState(false) const [hovered, setHovered] = useState(false)
const segment = useSelectedLayoutSegment() const segment = useSelectedLayoutSegment()
const isActived = Array.isArray(activeSegment) ? activeSegment.includes(segment!) : segment === activeSegment const isActived = Array.isArray(activeSegment) ? activeSegment.includes(segment!) : segment === activeSegment
...@@ -40,6 +42,7 @@ const Nav = ({ ...@@ -40,6 +42,7 @@ const Nav = ({
`}> `}>
<Link href={link}> <Link href={link}>
<div <div
onClick={() => setAppDetail()}
className={classNames(` className={classNames(`
flex items-center h-7 px-2.5 cursor-pointer rounded-[10px] flex items-center h-7 px-2.5 cursor-pointer rounded-[10px]
${isActived ? 'text-primary-600' : 'text-gray-500'} ${isActived ? 'text-primary-600' : 'text-gray-500'}
......
...@@ -7,8 +7,9 @@ import { debounce } from 'lodash-es' ...@@ -7,8 +7,9 @@ import { debounce } from 'lodash-es'
import Indicator from '../../indicator' import Indicator from '../../indicator'
import AppIcon from '@/app/components/base/app-icon' import AppIcon from '@/app/components/base/app-icon'
import { useAppContext } from '@/context/app-context' import { useAppContext } from '@/context/app-context'
import { useStore as useAppStore } from '@/app/components/app/store'
type NavItem = { export type NavItem = {
id: string id: string
name: string name: string
link: string link: string
...@@ -31,6 +32,7 @@ const itemClassName = ` ...@@ -31,6 +32,7 @@ const itemClassName = `
const NavSelector = ({ curNav, navs, createText, onCreate, onLoadmore }: INavSelectorProps) => { const NavSelector = ({ curNav, navs, createText, onCreate, onLoadmore }: INavSelectorProps) => {
const router = useRouter() const router = useRouter()
const { isCurrentWorkspaceManager } = useAppContext() const { isCurrentWorkspaceManager } = useAppContext()
const { setAppDetail } = useAppStore()
const handleScroll = useCallback(debounce((e) => { const handleScroll = useCallback(debounce((e) => {
if (typeof onLoadmore === 'function') { if (typeof onLoadmore === 'function') {
...@@ -70,7 +72,10 @@ const NavSelector = ({ curNav, navs, createText, onCreate, onLoadmore }: INavSel ...@@ -70,7 +72,10 @@ const NavSelector = ({ curNav, navs, createText, onCreate, onLoadmore }: INavSel
{ {
navs.map(nav => ( navs.map(nav => (
<Menu.Item key={nav.id}> <Menu.Item key={nav.id}>
<div className={itemClassName} onClick={() => router.push(nav.link)} title={nav.name}> <div className={itemClassName} onClick={() => {
setAppDetail()
router.push(nav.link)
}} title={nav.name}>
<div className='relative w-6 h-6 mr-2 bg-[#D5F5F6] rounded-[6px]'> <div className='relative w-6 h-6 mr-2 bg-[#D5F5F6] rounded-[6px]'>
<AppIcon size='tiny' icon={nav.icon} background={nav.icon_background}/> <AppIcon size='tiny' icon={nav.icon} background={nav.icon_background}/>
<div className='flex justify-center items-center absolute -right-0.5 -bottom-0.5 w-2.5 h-2.5 bg-white rounded'> <div className='flex justify-center items-center absolute -right-0.5 -bottom-0.5 w-2.5 h-2.5 bg-white rounded'>
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment