feat(user-account-merge): add account merge tracking and edX/MIT Learn user mapping #2474
feat(user-account-merge): add account merge tracking and edX/MIT Learn user mapping #2474rachellougee wants to merge 2 commits into
Conversation
…r user enrollments
There was a problem hiding this comment.
Pull request overview
Adds dbt models to support cross-platform identity tracking by deriving MITx Online account-merge redirects from enrollment audit history and exposing a stable edX.org ↔ MIT Learn crosswalk via MITx Online.
Changes:
- Introduces new staging models (and source definitions) for MITx Online course run/program enrollment audit logs.
- Adds
bridge_user_account_mergedimensional bridge table to track verified MITx Online account-merge redirects. - Adds
edxorg_mitlearn_user_mappingreporting model (with schema docs) for stable cross-platform user mapping.
Reviewed changes
Copilot reviewed 8 out of 8 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
| src/ol_dbt/models/staging/mitxonline/stg__mitxonline__app__postgres__courses_programenrollmentaudit.sql | New staging model for program enrollment audit history. |
| src/ol_dbt/models/staging/mitxonline/stg__mitxonline__app__postgres__courses_courserunenrollmentaudit.sql | New staging model for course run enrollment audit history. |
| src/ol_dbt/models/staging/mitxonline/_stg_mitxonline__models.yml | Adds schema docs/tests for the new staging audit models. |
| src/ol_dbt/models/staging/mitxonline/_mitxonline__sources.yml | Registers the new raw audit tables as dbt sources. |
| src/ol_dbt/models/reporting/edxorg_mitlearn_user_mapping.sql | New reporting crosswalk between edX.org and MIT Learn via MITx Online (+ merge redirects). |
| src/ol_dbt/models/reporting/_reporting__models.yml | Adds schema docs for the new reporting model. |
| src/ol_dbt/models/dimensional/bridge_user_account_merge.sql | New bridge table deriving merge redirects from audit history. |
| src/ol_dbt/models/dimensional/_bridge_tables.yml | Adds schema docs/tests for bridge_user_account_merge. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
…platform link logic
🔎 ol-dbt impact — column-level blast radius0 breaking, 0 warning, 5 info across 5 changed model(s). Details
Posted by |
KatelynGit
left a comment
There was a problem hiding this comment.
The builds ran clean, the SQL looked ok to me, but the definitions are odd- they don't look like full sentences, are a bit confusing, and often contain strange characters I'd probably just give them a read through before merging.
|
I just want to emphasize @KatelynGit's comments. The descriptions are meant to be read by our end data platform end users. I know these tables are bit esoteric, but we could at least start with a high-level general statement about what the table is for, before getting into the details about how they are constructed. Perhaps Claude can help. |
What are the relevant tickets?
https://github.com/mitodl/hq/issues/12430
Description (What does it do?)
bridge_user_account_link: tracks MITx Online account merges and cross-platform account links (edX.org), derived from enrollment/program audit history.edxorg_mitlearn_user_mapping: a standalone reporting table giving a stable edX.org ↔ MIT Learn crosswalk (both connect to MITx Online internally).A stable way to look up a learner's edX.org and MIT Learn accounts, since usernames/emails on both platforms can change over time.
How can this be tested?
Additional Context