feat(onboarding): add step 2

[mamadoudicko]

Screen.Recording.2023-10-03.at.16.52.53.mov

#1309

[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	Oct 3, 2023 4:28pm
quivr-strapi	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Oct 3, 2023 4:28pm
quivrapp	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Oct 3, 2023 4:28pm

[github-actions]

Risk Level 2 - /home/runner/work/quivr/quivr/frontend/app/chat/[chatId]/components/ChatDialogueArea/components/ChatDialogue/components/Onboarding/components/Step1.tsx

The code is generally clean and readable. However, there is a potential issue with the Link component. The download attribute is not supported in the Link component from next/link. You should use an a tag instead.

<a
  href=\"/documents/doc.pdf\"
  download
  target=\"_blank\"
  referrerPolicy=\"no-referrer\"
  onClick={() => changeStateTo(\"UPLOAD\")}
>
  <Button className=\"bg-black p-2 ml-2 rounded-full inline-flex\">
    <RiDownloadLine />
  </Button>
</a>

---

Risk Level 2 - /home/runner/work/quivr/quivr/frontend/app/chat/[chatId]/components/ChatDialogueArea/components/ChatDialogue/components/Onboarding/hooks/useStreamText.ts

The code is generally clean and readable. However, there is a potential performance issue with the setInterval function. If the text prop changes while the interval is running, it could lead to unexpected behavior. Consider clearing the interval when the text prop changes.

useEffect(() => {
  let messageInterval: NodeJS.Timeout;

  if (enabled) {
    messageInterval = setInterval(() => {
      // ...
    }, 30);
  }

  return () => {
    clearInterval(messageInterval);
  };
}, [text, enabled]);

---

🔗🕰️🧹

---

Powered by Code Review GPT