-
Notifications
You must be signed in to change notification settings - Fork 108
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 database action relation ship #7303
Comments
Any progress on this @fontanierh? |
What model are we talking about? AgentQueryTableConfigurationTable or something else? |
TBH this slipped a bit, I need to understand if this is still relevant (a bunch of related code has moved quite a bit these past weeks) |
The main one to fix was |
The one that matters is **ConfigurationTable in the context of data sources and this one is aligned with the retrieval approach: restrict with explicit check and deletes AFAICT. |
But maybe this is not a data source only convo? |
@flvndvd I think this is OK to close no ? |
Assigning to self, will do a final pass. |
We should set the right relation ships in our data base to prevent deleting data source views and vaults that would still be used by assistants.
The most likely forward path here is to set
ON DELETE RESTRICT
.First occurence to fix is reverting: #6676.
The text was updated successfully, but these errors were encountered: