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.

Sign up for GitHub

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

Jump to bottom

Engagement charts error when embedded #755

Closed
bmtcril opened this issue Apr 29, 2024 · 1 comment · Fixed by openedx/platform-plugin-aspects#40
Closed

Engagement charts error when embedded #755

bmtcril opened this issue Apr 29, 2024 · 1 comment · Fixed by openedx/platform-plugin-aspects#40
Assignees
@Ian2012

Comments

@bmtcril
Copy link
Contributor

bmtcril commented Apr 29, 2024 •
edited
Loading

Our embedding token limits course access by course_name, course_run, and org. When a chart query doesn't have those 3 fields it will error out. So we need to join in course_name, probably somewhere around here:

tutor-contrib-aspects/tutoraspects/templates/openedx-assets/queries/fact_problem_engagement.sql

Line 19 in 89adc0e

from {{ DBT_PROFILE_TARGET_DATABASE }}.fact_problem_engagement

The error presents as:

Missing columns: 'course_name' while processing query

And impacts problem engagement, page engagement, and video engagement currently.
@bmtcril
Copy link
Contributor Author

bmtcril commented Apr 30, 2024

openedx/platform-plugin-aspects#40 should close this

@Ian2012 Ian2012 linked a pull request Apr 30, 2024 that will close this issue
fix: allow to embed dashboards in xblock openedx/platform-plugin-aspects#40
Merged
7 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Ian2012 Ian2012

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

fix: allow to embed dashboards in xblock openedx/platform-plugin-aspects
2 participants
@Ian2012 @bmtcril