openedx / aspects-dbt

The dbt project for Open edX Aspects!
Apache License 2.0
2 stars 4 forks source link

feat: add enrollment timestamp to at-risk learners (FC-0051) #83

Closed SoryRawyer closed 3 months ago

SoryRawyer commented 3 months ago

This change adds the timestamp of when a learner enrolled in a course to the fact_student_status model. This field is then used in dim_at_risk_learners to support displaying a distribution of when at-risk learners enrolled in the course.

openedx-webhooks commented 3 months ago

Thanks for the pull request, @SoryRawyer! Please note that it may take us up to several weeks or months to complete a review and merge your PR.

Feel free to add as much of the following information to the ticket as you can:

All technical communication about the code itself will be done via the GitHub pull request interface. As a reminder, our process documentation is here.

Please let us know once your PR is ready for our review and all tests are green.

bmtcril commented 3 months ago

This is the most recent enrollment time, right? So when a user changes status (upgrades to certificate or unenrolls and re-enrolls) the clock resets? We should probably include that detail in the model docs for this and fact_enrollment_status.

SoryRawyer commented 3 months ago

That's a good point. Yes, it should be the most recent enrollment action timestamp.

openedx-webhooks commented 3 months ago

@SoryRawyer 🎉 Your pull request was merged! Please take a moment to answer a two question survey so we can improve your experience in the future.

bmtcril commented 3 months ago

Released as v3.22.0, tutor plugin PR is here: https://github.com/openedx/tutor-contrib-aspects/pull/763