-
-
Notifications
You must be signed in to change notification settings - Fork 3.6k
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
fix(chats): now in order and with a little bonus ;) #1200
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
Risk Level 2 - /home/runner/work/quivr/quivr/frontend/lib/utils.ts The added functions in this file are used to check if a given date is today, yesterday, within the last 7 days, or within the last 30 days. These functions seem to be implemented correctly, but it's recommended to write unit tests for these functions to ensure they work as expected. Risk Level 2 - /home/runner/work/quivr/quivr/frontend/app/chat/components/ChatsList/index.tsx The changes in this file involve filtering chats into different groups based on their creation time. This could potentially introduce bugs if the date filtering functions do not work as expected. However, without seeing these functions, it's hard to say for sure. It's recommended to thoroughly test these changes to ensure the filtering works correctly. 📅🔍✅ Powered by Code Review GPT |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sorry but no /* eslint-disable */
Chats ordered & with Today, Yesterday, Last 7 days and more ;)