openedx / aspects-dbt

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

fix: use latest xapi base MV definition (FC-0033) #38

Closed SoryRawyer closed 10 months ago

SoryRawyer commented 10 months ago

While reviewing the refactor to remove the MVs from our alembic migrations, I noticed that I missed the last update to the xapi_events_all_parsed_mv definition. This PR brings the dbt implementation in line with the current alembic-managed version.

openedx-webhooks commented 10 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.

openedx-webhooks commented 10 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.