-
-
Notifications
You must be signed in to change notification settings - Fork 658
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: create client_applications_usage table migration #4521
feat: create client_applications_usage table migration #4521
Conversation
Signed-off-by: andreas-unleash <andreas@getunleash.ai>
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
` | ||
CREATE TABLE IF NOT EXISTS client_applications_usage ( | ||
app_name VARCHAR(255) REFERENCES client_applications(app_name) ON DELETE CASCADE, | ||
project VARCHAR(255) REFERENCES projects(id) NOT NULL default 'default', |
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.
I think we can not put foreign key on this, because we also have *. Same for environment
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.
Also, can there be instances where default project does not exist. I am not sure, but looks potentially dangerous.
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.
fair enough - default project and environment are created by our migrations. Good point on the *
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.
Actually we could preserve the FK, but keep it nullable. And then null means all projects.
Signed-off-by: andreas-unleash <andreas@getunleash.ai>
Creates client_applications_usage table