Skip to content

Commit 3a48980

Browse files
committed
fix(kb): fix kb navigation URLs
1 parent b42c43a commit 3a48980

File tree

5 files changed

+22
-11
lines changed

5 files changed

+22
-11
lines changed

apps/sim/app/workspace/[workspaceId]/knowledge/[id]/[documentId]/components/document-loading.tsx

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,7 @@
11
'use client'
22

33
import { Plus, Search } from 'lucide-react'
4+
import { useParams } from 'next/navigation'
45
import { Button } from '@/components/ui/button'
56
import { useSidebarStore } from '@/stores/sidebar/store'
67
import { KnowledgeHeader } from '../../../components/knowledge-header/knowledge-header'
@@ -18,19 +19,21 @@ export function DocumentLoading({
1819
documentName,
1920
}: DocumentLoadingProps) {
2021
const { mode, isExpanded } = useSidebarStore()
22+
const params = useParams()
23+
const workspaceId = params?.workspaceId as string
2124
const isSidebarCollapsed =
2225
mode === 'expanded' ? !isExpanded : mode === 'collapsed' || mode === 'hover'
2326

2427
const breadcrumbs = [
2528
{
2629
id: 'knowledge-root',
2730
label: 'Knowledge',
28-
href: '/knowledge',
31+
href: `/workspace/${workspaceId}/knowledge`,
2932
},
3033
{
3134
id: `knowledge-base-${knowledgeBaseId}`,
3235
label: knowledgeBaseName,
33-
href: `/knowledge/${knowledgeBaseId}`,
36+
href: `/workspace/${workspaceId}/knowledge/${knowledgeBaseId}`,
3437
},
3538
{
3639
id: `document-${knowledgeBaseId}-${documentName}`,

apps/sim/app/workspace/[workspaceId]/knowledge/[id]/[documentId]/document.tsx

Lines changed: 6 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -12,6 +12,7 @@ import {
1212
Trash2,
1313
X,
1414
} from 'lucide-react'
15+
import { useParams } from 'next/navigation'
1516
import { Button } from '@/components/ui/button'
1617
import { Checkbox } from '@/components/ui/checkbox'
1718
import { Tooltip, TooltipContent, TooltipTrigger } from '@/components/ui/tooltip'
@@ -54,6 +55,7 @@ export function Document({
5455
}: DocumentProps) {
5556
const { mode, isExpanded } = useSidebarStore()
5657
const { getCachedKnowledgeBase, getCachedDocuments } = useKnowledgeStore()
58+
const { workspaceId } = useParams()
5759

5860
const isSidebarCollapsed =
5961
mode === 'expanded' ? !isExpanded : mode === 'collapsed' || mode === 'hover'
@@ -170,10 +172,10 @@ export function Document({
170172
const effectiveDocumentName = document?.filename || documentName || 'Document'
171173

172174
const breadcrumbs = [
173-
{ label: 'Knowledge', href: '/knowledge' },
175+
{ label: 'Knowledge', href: `/workspace/${workspaceId}/knowledge` },
174176
{
175177
label: effectiveKnowledgeBaseName,
176-
href: `/knowledge/${knowledgeBaseId}`,
178+
href: `/workspace/${workspaceId}/knowledge/${knowledgeBaseId}`,
177179
},
178180
{ label: effectiveDocumentName },
179181
]
@@ -360,10 +362,10 @@ export function Document({
360362

361363
if (combinedError && !isLoadingChunks) {
362364
const errorBreadcrumbs = [
363-
{ label: 'Knowledge', href: '/knowledge' },
365+
{ label: 'Knowledge', href: `/workspace/${workspaceId}/knowledge` },
364366
{
365367
label: effectiveKnowledgeBaseName,
366-
href: `/knowledge/${knowledgeBaseId}`,
368+
href: `/workspace/${workspaceId}/knowledge/${knowledgeBaseId}`,
367369
},
368370
{ label: 'Error' },
369371
]

apps/sim/app/workspace/[workspaceId]/knowledge/[id]/base.tsx

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -743,7 +743,7 @@ export function KnowledgeBase({
743743
{
744744
id: 'knowledge-root',
745745
label: 'Knowledge',
746-
href: '/knowledge',
746+
href: `/workspace/${workspaceId}/knowledge`,
747747
},
748748
{
749749
id: `knowledge-base-${id}`,
@@ -762,7 +762,7 @@ export function KnowledgeBase({
762762
{
763763
id: 'knowledge-root',
764764
label: 'Knowledge',
765-
href: '/knowledge',
765+
href: `/workspace/${workspaceId}/knowledge`,
766766
},
767767
{
768768
id: 'error',

apps/sim/app/workspace/[workspaceId]/knowledge/[id]/components/knowledge-base-loading/knowledge-base-loading.tsx

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,7 @@
11
'use client'
22

33
import { Search } from 'lucide-react'
4+
import { useParams } from 'next/navigation'
45
import { Button } from '@/components/ui/button'
56
import { useSidebarStore } from '@/stores/sidebar/store'
67
import { KnowledgeHeader } from '../../../components/knowledge-header/knowledge-header'
@@ -12,14 +13,16 @@ interface KnowledgeBaseLoadingProps {
1213

1314
export function KnowledgeBaseLoading({ knowledgeBaseName }: KnowledgeBaseLoadingProps) {
1415
const { mode, isExpanded } = useSidebarStore()
16+
const params = useParams()
17+
const workspaceId = params?.workspaceId as string
1518
const isSidebarCollapsed =
1619
mode === 'expanded' ? !isExpanded : mode === 'collapsed' || mode === 'hover'
1720

1821
const breadcrumbs = [
1922
{
2023
id: 'knowledge-root',
2124
label: 'Knowledge',
22-
href: '/knowledge',
25+
href: `/workspace/${workspaceId}/knowledge`,
2326
},
2427
{
2528
id: 'knowledge-base-loading',

apps/sim/app/workspace/[workspaceId]/knowledge/components/base-overview/base-overview.tsx

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -3,6 +3,7 @@
33
import { useState } from 'react'
44
import { Check, Copy, LibraryBig } from 'lucide-react'
55
import Link from 'next/link'
6+
import { useParams } from 'next/navigation'
67

78
interface BaseOverviewProps {
89
id?: string
@@ -13,12 +14,14 @@ interface BaseOverviewProps {
1314

1415
export function BaseOverview({ id, title, docCount, description }: BaseOverviewProps) {
1516
const [isCopied, setIsCopied] = useState(false)
17+
const params = useParams()
18+
const workspaceId = params?.workspaceId as string
1619

1720
// Create URL with knowledge base name as query parameter
18-
const params = new URLSearchParams({
21+
const searchParams = new URLSearchParams({
1922
kbName: title,
2023
})
21-
const href = `/knowledge/${id || title.toLowerCase().replace(/\s+/g, '-')}?${params.toString()}`
24+
const href = `/workspace/${workspaceId}/knowledge/${id || title.toLowerCase().replace(/\s+/g, '-')}?${searchParams.toString()}`
2225

2326
const handleCopy = async (e: React.MouseEvent) => {
2427
e.preventDefault()

0 commit comments

Comments
 (0)