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

fix: delete brain on users click in brains management #1638

Merged
merged 4 commits into from
Nov 16, 2023

Conversation

gozineb
Copy link
Contributor

@gozineb gozineb commented Nov 16, 2023

Issue: #1644

Description

Fix subscription routes on get users
Revert changes deleting the function to get user email from its id in SQL

https://www.loom.com/share/46534965a83f4296898495ac370b3451?sid=7f582838-32f6-4812-960e-c80d71517a6d

@dosubot dosubot bot added the area: backend Related to backend functionality or under the /backend directory label Nov 16, 2023
Copy link

vercel bot commented Nov 16, 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 Nov 16, 2023 10:13am
quivr-strapi ✅ Ready (Inspect) Visit Preview 💬 Add feedback Nov 16, 2023 10:13am
quivrapp ✅ Ready (Inspect) Visit Preview 💬 Add feedback Nov 16, 2023 10:13am

@mamadoudicko mamadoudicko merged commit 18946ad into main Nov 16, 2023
gozineb pushed a commit that referenced this pull request Nov 16, 2023
🤖 I have created a release *beep* *boop*
---


## 0.0.114 (2023-11-16)

## What's Changed
* feat: add api brain creation frontend by @mamadoudicko in
#1631
* refactor: add modules folder by @gozineb in
#1633
* feat: update settings tab add api brain definition by @mamadoudicko in
#1635
* feat: add public api brain subscription by @mamadoudicko in
#1636
* fix: delete brain on users click in brains management by @gozineb in
#1638


**Full Changelog**:
v0.0.113...v0.0.114

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).
Copy link

sentry-io bot commented Nov 25, 2023

Suspect Issues

This pull request was deployed and Sentry observed the following issues:

  • ‼️ APIError: Database connection error. Retrying the connection. /brains/{brain_id}/users View Issue

Did you find this useful? React with a 👍 or 👎

coolCatalyst added a commit to coolCatalyst/quivr that referenced this pull request Jun 1, 2024
🤖 I have created a release *beep* *boop*
---


## 0.0.114 (2023-11-16)

## What's Changed
* feat: add api brain creation frontend by @mamadoudicko in
QuivrHQ/quivr#1631
* refactor: add modules folder by @gozineb in
QuivrHQ/quivr#1633
* feat: update settings tab add api brain definition by @mamadoudicko in
QuivrHQ/quivr#1635
* feat: add public api brain subscription by @mamadoudicko in
QuivrHQ/quivr#1636
* fix: delete brain on users click in brains management by @gozineb in
QuivrHQ/quivr#1638


**Full Changelog**:
QuivrHQ/quivr@v0.0.113...v0.0.114

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: backend Related to backend functionality or under the /backend directory
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants