Skip to content

dbt docs: clarify tenant part of student surrogate keys#177

Merged
rlittle08 merged 3 commits into
mainfrom
hotfix/k_student_dbt_docs
Jul 18, 2025
Merged

dbt docs: clarify tenant part of student surrogate keys#177
rlittle08 merged 3 commits into
mainfrom
hotfix/k_student_dbt_docs

Conversation

@rlittle08
Copy link
Copy Markdown
Collaborator

@rlittle08 rlittle08 commented Jul 13, 2025

hotfix/k_student_dbt_docs

PR Merge Priority:

  • Low
  • Medium
  • High

Changes to existing files:

  • Update dim_student.yaml to be more precise on surrogate key defs for commonly referenced k_student and k_student_xyear.

edu_wh PR Review Checklist:

Make sure the following have been completed before approving this PR:

  • Description of changes has been added to Unreleased section of CHANGELOG.md. Add under ## New Features for features, etc.
  • If a new configuration xwalk was added:
    • The code is written such that the xwalk is optional (preferred), and this behavior was tested, OR
    • The code is written such that the xwalk is required, and the required xwalk is added to edu_project_template, and this PR is flagged as breaking change (not for patch release)
    • A description for the new xwalk has been added to EDU documentation site here
  • If a new configuration variable was added:
    • The code is written such that the variable is optional (preferred), and this behavior was tested, OR
    • The code is written such that the variable is required, and a default value was added to edu_project_template, and this PR is flagged as breaking change (not for patch release)
    • A description for the new variable has been added to EDU documentation site here
  • Reviewer confirms the grain of all tables are unchanged, OR any changes are expected, communicated, and this PR is flagged as a breaking change (not for patch release)

@rlittle08 rlittle08 changed the title clarify tenant part of student surrogate keys dbt docs: clarify tenant part of student surrogate keys Jul 16, 2025
@rlittle08 rlittle08 merged commit d7e8709 into main Jul 18, 2025
@rlittle08 rlittle08 deleted the hotfix/k_student_dbt_docs branch July 18, 2025 20:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants