-
Notifications
You must be signed in to change notification settings - Fork 16
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
Operator Dashboard Enhancements #745
Conversation
...ates/aspects/build/aspects-superset/openedx-assets/assets/dashboards/Operator_Dashboard.yaml
Outdated
Show resolved
Hide resolved
tutoraspects/templates/aspects/apps/superset/pythonpath/create_assets_utils.py
Outdated
Show resolved
Hide resolved
tutoraspects/plugin.py
Outdated
@@ -334,7 +334,7 @@ | |||
# For now we are pulling this from github, which should allow maximum | |||
# flexibility for forking, running branches, specific versions, etc. | |||
("DBT_REPOSITORY", "https://github.com/openedx/aspects-dbt"), | |||
("DBT_BRANCH", "v3.19.0"), | |||
("DBT_BRANCH", "bmtcril/operator_dash_enhancements"), |
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.
This needs to be updated once the latest aspects-dbt is released
Deleted files were detected as using removed datasets using the new process.
59cbdda
to
321b2c4
Compare
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.
LGTM, tested locally
I'm still waiting for tutor init to finish so I can confirm that the change from |
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.
looks good!
This PR includes a number of bug fixes and enhancements, you may want to review by commit.
Requires openedx/aspects-dbt#74 to merge and release before this can be merged.