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

feat: Remove chat id from chat list #1031

Merged
merged 2 commits into from
Aug 24, 2023

Conversation

ChloeMouret
Copy link
Contributor

@ChloeMouret ChloeMouret commented Aug 24, 2023

  • update hover color : blue for modifying and red for delete
  • remove chat id from chat lists

Description

Please include a summary of the changes and the related issue. Please also include relevant motivation and context.

Checklist before requesting a review

Please delete options that are not relevant.

  • My code follows the style guidelines of this project
  • I have performed a self-review of my code
  • I have commented hard-to-understand areas
  • I have ideally added tests that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes
  • Any dependent changes have been merged

Screenshots (if appropriate):

Capture d’écran 2023-08-24 à 15 37 55 Capture d’écran 2023-08-24 à 15 55 15 Capture d’écran 2023-08-24 à 15 55 21

@vercel
Copy link

vercel bot commented Aug 24, 2023

@ChloeMouret is attempting to deploy a commit to the Quivr-app Team on Vercel.

A member of the Team first needs to authorize it.

@vercel
Copy link

vercel bot commented Aug 24, 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 Aug 24, 2023 5:42pm
quivrapp ✅ Ready (Inspect) Visit Preview 💬 Add feedback Aug 24, 2023 5:42pm

@ChloeMouret ChloeMouret changed the title Remove chat id from chat list feat: Remove chat id from chat list Aug 24, 2023
<button className="p-0" type="button" onClick={handleEditNameClick}>
<div className="opacity-0 group-hover:opacity-100 flex items-center justify-center bg-gradient-to-l from-white dark:from-black to-transparent z-10 transition-opacity">
<button
className="p-0 hover:text-blue-700"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice hover addition !

Copy link
Contributor

@gozineb gozineb left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @ChloeMouret for the clean work 🥇

@StanGirard
Copy link
Collaborator

So cool :)

@StanGirard StanGirard merged commit d36217b into QuivrHQ:main Aug 24, 2023
4 of 8 checks passed
ccaztillol added a commit to ccaztillol/quivr that referenced this pull request Aug 24, 2023
feat: Remove chat id from chat list (QuivrHQ#1031)
StanGirard pushed a commit that referenced this pull request Sep 12, 2023
* update hover color : blue for modifying and red for delete

* remove chat id from chat lists
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants