TN #231: Feature/section characteristics in dim course section - #267
Merged
rlittle08 merged 2 commits intoAug 4, 2026
Merged
Conversation
- left outer join -> left join in bld_ef3__section__wide_section_characteristics - Remove outer=True from json_flatten; sections without characteristics are NULL in dim_course_section via left join, matching course-level behavior - Exclude tenant_code/api_year in accordion_columns for section_chars in dim_course_section - Remove stale todo comment from dim_course_section - Register bld_ef3__section__wide_section_characteristics in _edfi_3__courses.yml with core tag - Add section_characteristics_array column entry to dim_course_section.yml - Expand Important Business Rules to mention xwalk_section_characteristics and section-level is_* columns Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
AngelicaLastra
approved these changes
Aug 4, 2026
rlittle08
deleted the
feature/section_characteristics_in_dim_course_section
branch
August 4, 2026 17:27
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This is a copy of TN #231 -- with a couple cosmetic updates plus a fix for rls. Only opening from here bc I don't have ability to commit updates or open PRs from TN's fork
Thank you @smckee-tnedu for this contribution