Skip to content

fix: eligibility rules - #110

Merged
mdiapenabc merged 1 commit into
devfrom
fix/eligibility-fixes
Mar 3, 2026
Merged

fix: eligibility rules#110
mdiapenabc merged 1 commit into
devfrom
fix/eligibility-fixes

Conversation

@mdiapenabc

@mdiapenabc mdiapenabc commented Mar 3, 2026

Copy link
Copy Markdown
Collaborator

Description

Fixes several bugs in the eligibility decision tree

  • Step 2 to Step 8 transition
  • Step 9 missing the null csa_status check
  • Timezone issues

How to Test

# Install dependencies
npm install && cd backend && npm install

# Run tests (should all pass — no new service logic yet)
cd backend && npm run test

# Verify migration applies cleanly (requires local Postgres)
Run migration: docker compose down -v & docker compose up 
Run data ingestion: npm run job:data-ingestion

@mdiapenabc
mdiapenabc force-pushed the fix/eligibility-fixes branch from 8cfb0a3 to 2587c8d Compare March 3, 2026 11:10
@mdiapenabc
mdiapenabc requested a review from saifrazabc March 3, 2026 11:40

@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 eligibility rules & date format fixies.

@mdiapenabc
mdiapenabc requested a review from kayboy-007 March 3, 2026 11:46
@mdiapenabc
mdiapenabc merged commit bc2e02b into dev Mar 3, 2026
15 checks passed
@mdiapenabc
mdiapenabc deleted the fix/eligibility-fixes branch March 3, 2026 11:47
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