Skip to content

feat: update eligibility rules - #152

Merged
mdiapenabc merged 2 commits into
devfrom
feat/eligibility-rules-update
Mar 18, 2026
Merged

feat: update eligibility rules#152
mdiapenabc merged 2 commits into
devfrom
feat/eligibility-rules-update

Conversation

@mdiapenabc

@mdiapenabc mdiapenabc commented Mar 18, 2026

Copy link
Copy Markdown
Collaborator

Description

  • Updates eligibility rules to handle eligible_tbd and not_eligible_ip_tbd statuses in outcome steps
  • Add date filtering and Ended/Closed fallback logic to Step 3, and fixes a frontend date abbreviation inconsistency.

What Changes

  • Unprotect eligible_tbd and not_eligible_ip_tbd
  • Step 7: eligible_tbd TO eligible, not_eligible_ip_tbd TO in_pay
  • Step 9: eligible_tbd / not_eligible_ip_tbd TO not_eligible_out_of_pay
  • Step 10: eligible_tbd / not_eligible_in_pay / not_eligible_ip_tbd TO over_18
  • Step 3: Add date filtering — Active/Interrupted placements must have startDate prior to current month; Ended/Closed fallback checks endDate in previous month; Ended/Closed Non-Placements now route to Step 4 instead of Step 8
  • Frontend: Normalize month abbreviation with Locale en-US instead of 'en-CA' to fix Sept vs Sep inconsistency across CLDR versions

@mdiapenabc mdiapenabc changed the title update eigibility rules feat: update eigibility rules Mar 18, 2026
@mdiapenabc mdiapenabc changed the title feat: update eigibility rules feat: update eligibility rules Mar 18, 2026
@mdiapenabc
mdiapenabc force-pushed the feat/eligibility-rules-update branch from 7f55225 to ddda106 Compare March 18, 2026 07:25

@saifrazabc saifrazabc left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approving for update eligibility rules.

@mdiapenabc
mdiapenabc merged commit c32bd0c into dev Mar 18, 2026
15 checks passed
@mdiapenabc
mdiapenabc deleted the feat/eligibility-rules-update branch March 18, 2026 07:38
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