openedx / aspects-dbt

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

feat: add fact_current_enrollments model #101

Closed Ian2012 closed 2 months ago

Ian2012 commented 2 months ago

Description

This PR adds a model to get the latest enrollment status of a learner but with extra course and learner information.

partially fixes: https://github.com/openedx/wg-build-test-release/issues/368

Changes need to be applied to Superset.

openedx-webhooks commented 2 months ago

Thanks for the pull request, @Ian2012! 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.

Ian2012 commented 2 months ago

It's the same data but with the joined course and user data, however, the MV will only be updated on changes to the enrollments, so if I join user pii with fact_enrollment_status the data will be outdated. fact_enrollment_status stored the latest enrollment status however the data is eventually consistent so I will add a group by clause

openedx-webhooks commented 2 months ago

@Ian2012 Even though your pull request wasn’t merged, please take a moment to answer a two question survey so we can improve your experience in the future.