Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore(main): release 0.0.88 #1322

Conversation

StanGirard
Copy link
Collaborator

@StanGirard StanGirard commented Oct 4, 2023

🤖 I have created a release beep boop

0.0.88 (2023-10-05)

What's Changed

Full Changelog: v0.0.87...v0.0.88


This PR was generated with Release Please. See documentation.

@vercel
Copy link

vercel bot commented Oct 4, 2023

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 5, 2023 9:36am
quivr-strapi ✅ Ready (Inspect) Visit Preview 💬 Add feedback Oct 5, 2023 9:36am
quivrapp ✅ Ready (Inspect) Visit Preview 💬 Add feedback Oct 5, 2023 9:36am

@github-actions
Copy link
Contributor

github-actions bot commented Oct 4, 2023

Risk Level 2 - /home/runner/work/quivr/quivr/frontend/app/chat/components/ChatsList/hooks/useChatsList.ts

  1. The fetchAllChats function is returning the reversed response. This might not be the expected behavior as it changes the order of the chats. If the order is important, consider removing the reverse function.
const response = await getChats();
return response;

Risk Level 2 - /home/runner/work/quivr/quivr/frontend/app/chat/[chatId]/components/ActionsBar/components/KnowledgeToFeed/hooks/useFeedBrainInChat.ts

  1. The UUID import from crypto is not used in the code, consider removing it to improve readability.
  2. The useParams import from next/navigation is incorrect. It should be imported from next/router.
  3. The catch block in the feedBrain function is logging the error to the console and publishing a toast notification with the error message. This could potentially expose sensitive information to the end user. Consider logging the error for debugging purposes but provide a generic error message to the user.
} catch (e) {
  console.error(e);
  publish({
    variant: \"danger\",
    text: t(\"errorOccurred\"),
  });
}

🔄🧹🔒


Powered by Code Review GPT

@StanGirard StanGirard force-pushed the release-please--branches--main--components--release-please-action branch from 1ad2f38 to 9625475 Compare October 5, 2023 09:32
@gozineb gozineb merged commit 408aa8f into main Oct 5, 2023
6 checks passed
@StanGirard
Copy link
Collaborator Author

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants