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

Update real_time_dashboard_usage_by_dashboard.sql #1

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

shahmohamadi
Copy link

change "_" to "." in column names

change "_" to "." in column names
@robertocommit
Copy link
Collaborator

@shahmohamadi thanks for this! Is it schema metabase better than public?

@shahmohamadi
Copy link
Author

@shahmohamadi thanks for this! Is it schema metabase better than public?

I have a "metabase" schema on the database, but that's not so important. two column names need editing I think:

public.view_log_user_id ==> public.view_log.user_id
public.core_user_id ==> public.core_user.id

I got some partial on some of your other scripts in this repo too that would comment soon

@robertocommit
Copy link
Collaborator

Well, I think it makes sense to have metabase instead of public, much more straightforward.

If you feel to change the schema also on all other queries, that would be amazing!

Last but not least, I need to add in the README how to connect metabase database (at the moment you can only do it if you self host metabase, and not with the premium version)

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.

None yet

2 participants