feat: add polling for pending notifications

[mamadoudicko]

• Add polling logic on pending notifications on chat page

Screen.Recording.2023-09-12.at.16.46.23.mov

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated (UTC)
docs	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Sep 12, 2023 3:49pm
quivrapp	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Sep 12, 2023 3:49pm

[github-actions]

Risk Level 2 - /home/runner/work/quivr/quivr/frontend/app/chat/[chatId]/components/ActionsBar/ActionsBar.tsx

The changes in this file involve the addition of a new state variable setHasPendingRequests and its usage in the useKnowledgeUploader hook. This seems to be a standard operation in React and should not pose any risk. However, it's important to ensure that the state is managed correctly to avoid potential bugs.

---

Risk Level 3 - /home/runner/work/quivr/quivr/frontend/app/chat/[chatId]/hooks/useSelectedChatPage.ts

The changes in this file involve the addition of a new query for fetching notifications and its usage in a useEffect hook. This could potentially introduce bugs if not handled correctly. It's important to ensure that error handling is done properly and that the query is used in the correct places.

---

Risk Level 3 - /home/runner/work/quivr/quivr/frontend/app/chat/[chatId]/components/ActionsBar/hooks/useKnowledgeUploader.ts

The changes in this file involve the addition of a new function fetchNotifications and its usage in various places. This could potentially introduce bugs if not handled correctly. It's important to ensure that error handling is done properly and that the function is called in the correct places.

---

📁🐛⚠️

---

Powered by Code Review GPT