fix(frontend): don t load chat items on search (#2036)

# Description

Please include a summary of the changes and the related issue. Please
also include relevant motivation and context.

## Checklist before requesting a review

Please delete options that are not relevant.

- [ ] My code follows the style guidelines of this project
- [ ] I have performed a self-review of my code
- [ ] I have commented hard-to-understand areas
- [ ] I have ideally added tests that prove my fix is effective or that
my feature works
- [ ] New and existing unit tests pass locally with my changes
- [ ] Any dependent changes have been merged

## Screenshots (if appropriate):
This commit is contained in:
Antoine Dewez 2024-01-20 13:23:38 -08:00 committed by GitHub
parent 0a3b06b4eb
commit a99d23cf83
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
2 changed files with 14 additions and 5 deletions

View File

@ -7,12 +7,14 @@ import { useNotificationApi } from "@/lib/api/notification/useNotificationApi";
import { useChatContext } from "@/lib/context";
import { useKnowledgeToFeedContext } from "@/lib/context/KnowledgeToFeedProvider/hooks/useKnowledgeToFeedContext";
import { useChatInput } from "../components/ActionsBar/components/ChatInput/hooks/useChatInput";
import { getChatNotificationsQueryKey } from "../utils/getChatNotificationsQueryKey";
import { getMessagesFromChatItems } from "../utils/getMessagesFromChatItems";
import { getNotificationsFromChatItems } from "../utils/getNotificationsFromChatItems";
// eslint-disable-next-line @typescript-eslint/explicit-module-boundary-types
export const useChatNotificationsSync = () => {
const { message } = useChatInput()
const { setMessages, setNotifications, notifications } = useChatContext();
const { getChatItems } = useChatApi();
const { getChatNotifications } = useNotificationApi();
@ -65,10 +67,11 @@ export const useChatNotificationsSync = () => {
return;
}
const chatItems = await getChatItems(chatId);
setMessages(getMessagesFromChatItems(chatItems));
setNotifications(getNotificationsFromChatItems(chatItems));
if (!message) {
const chatItems = await getChatItems(chatId);
setMessages(getMessagesFromChatItems(chatItems));
setNotifications(getNotificationsFromChatItems(chatItems));
}
};
void fetchHistory();
}, [chatId]);

View File

@ -1,6 +1,6 @@
"use client";
import { usePathname, useRouter } from "next/navigation";
import { ReactNode, useEffect } from "react";
import { ReactNode, useEffect, useState } from "react";
import { KnowledgeToFeedProvider } from "@/lib/context";
@ -11,13 +11,19 @@ interface LayoutProps {
const Layout = ({ children }: LayoutProps): JSX.Element => {
const pathname = usePathname();
const router = useRouter();
const [isLoading, setIsLoading] = useState(true);
useEffect(() => {
if (pathname === '/chat') {
router.push('/search');
} else {
setIsLoading(false);
}
}, [pathname, router]);
if (isLoading) {
return <></>
}
return (
<KnowledgeToFeedProvider>