feat: add chat config modal

[mamadoudicko]

Description

• Add brain config overwriting from chat config

Related issue: #744

[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	Aug 1, 2023 7:41am
quivrapp	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Aug 1, 2023 7:41am

[github-actions]

LOGAF Level 3 - /home/runner/work/quivr/quivr/frontend/app/chat/[chatId]/components/ChatInput/components/ConfigModal/hooks/useConfigModal.ts

1. The useEffect hooks could benefit from more comments explaining their purpose.
2. The handleSubmit function could be improved by adding error handling for when chatId is undefined.

Example:

const handleSubmit = (e: FormEvent) => {
  e.preventDefault();
  if (chatId === undefined) {
    // Handle error
    return;
  }
  ...
};

---

LOGAF Level 3 - /home/runner/work/quivr/quivr/frontend/lib/components/NavBar/components/NavItems/index.tsx

1. The NavItems component could be improved by adding a more descriptive fallback for when isUserLoggedIn is false. Currently, it only shows Github and Discord links which might not be informative to the user.
2. The NavItems component could be improved by adding error handling for when session is undefined.

Example:

if (session === undefined) {
  // Handle error
  return;
}

---

LOGAF Level 3 - /home/runner/work/quivr/quivr/backend/core/repository/brain_subscription/resend_invitation_email.py

1. The resend.api_key is being set in the function. This should be set in a configuration file or environment variable.
2. The resend_invitation_email function is doing too much. Consider separating the email body creation into a separate function.

Example:

resend.api_key = os.getenv('RESEND_API_KEY')

---

💬🔧🔑

---

Powered by Code Review GPT