Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

draft of optionally not requiring stu_ed_org to reach dim_student #148

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

ejoranlienea
Copy link
Contributor

Concept: for some downstream reporting purposes, it is confusing for a record not to be in, e.g. 'enrollment' because the student is not in stuEdOrg -- this is an implicit requirement that is hard for folks to understand. We may decide to remove this logic and always left join in the future, but this branch puts it behind an optional flag to let us try it out before committing.

Concept: for some downstream reporting purposes, it is confusing for a
record not to be in, e.g. 'enrollment' because the student is not in
stuEdOrg -- this is an implicit requirement that is hard for folks to
understand. We may decide to remove this logic and _always_ left join in
the future, but this branch puts it behind an optional flag to let us
try it out before committing.
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.

1 participant