feat: upgrade button in user settings

[matthieujacq]

Description

Epic: #1429
User Story: #1431

• Add an upgrade button in user settings.
• Remove hover links on sidebar buttons (otherwise the link could partially hide the button)

Screenshots (if appropriate):

[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 25, 2023 10:42am
quivr-strapi	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Oct 25, 2023 10:42am
quivrapp	❌ Failed (Inspect)			Oct 25, 2023 10:42am

[github-actions]

Risk Level 2 - /home/runner/work/quivr/quivr/frontend/app/user/page.tsx

The is_premium variable is used before it is defined. This could potentially lead to a ReferenceError if userData is undefined or null. Consider moving the definition of is_premium after the check for session.

if (!session) {
  redirectToLogin();
}

const { userData } = useUserData();
const is_premium = userData?.is_premium;

---

Risk Level 2 - /home/runner/work/quivr/quivr/frontend/lib/components/Sidebar/components/SidebarFooter/components/SidebarFooterButton.tsx

The onClick prop is being overwritten if href is defined. This could lead to unexpected behavior if a parent component passes an onClick prop. Consider calling both functions if both href and onClick are defined.

if (href !== undefined) {
  const originalOnClick = onClick;
  onClick = () => {
    void router.push(href);
    originalOnClick?.();
  };
}

---

🚧🔍🐛

---

Powered by Code Review GPT