Skip to content

Episode Calculations - #6999

Open
eanders wants to merge 1 commit into
ea-enrollment-rollup-performancefrom
ea-enrollment-episode-calculation
Open

Episode Calculations#6999
eanders wants to merge 1 commit into
ea-enrollment-rollup-performancefrom
ea-enrollment-episode-calculation

Conversation

@eanders

@eanders eanders commented Sep 11, 2026

Copy link
Copy Markdown
Contributor

Merging this PR

  • use the squash-merge strategy for PRs targeting main

Description

During the performance work on the enrollments roll-up a few issues were identified with the calcuation of "new episode". This attempts to fix:

  • Lack of documentation
  • Duplicate episode markers on duplicate enrollments
  • Divergence from official the HUD definition

Type of change

  • Bug fix

Checklist before requesting review

  • I performed a self-review of my code
  • I ran the OP review skill
  • I ran the code that is being changed under ideal conditions, and it doesn't fail
  • If adding a new endpoint / exposing data in a new way, I have:
    • ensured the API can't leak data from other data sources
    • ensured this does not introduce N+1s
    • ensured permissions and visibility checks are performed in the right places
  • Any major architectural changes are supported by an approved ADR (Architectural Decision Record)
  • I updated the documentation (or not applicable)
  • I added spec tests (or not applicable)
  • I provided testing instructions in this PR or the related issue (or not applicable)

…n with the HUD spec and cleanup duplicate new episodes in the enrollment roll-up
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