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

fix: Use enrolled date if last visited is not set #823

Merged
merged 1 commit into from
May 23, 2024
+3 −2
Merged

fix: Use enrolled date if last visited is not set #823

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
5 changes: 3 additions & 2 deletions tutoraspects/templates/openedx-assets/queries/fact_learner_summary.sql
View file
Original file line number Diff line number Diff line change
@@ -31,7 +31,8 @@ with
fss.grade_bucket as grade_bucket,
fss.username as username,
fss.name as name,
fss.email as email
fss.email as email,
fss.enrolled_at as enrolled_at
from {{DBT_PROFILE_TARGET_DATABASE}}.fact_student_status fss
where
1=1
@@ -53,7 +54,7 @@ select
fss.name as name,
fss.email as email,
fes.max_emission_time as emission_time,
let.last_visited as last_visited
GREATEST(let.last_visited, fss.enrolled_at) as last_visited
from student_status fss
left join
enrollment_status fes