-
Notifications
You must be signed in to change notification settings - Fork 6
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
2024 Transfer assessment #848
Conversation
Moving to draft until we have timing of launch determined. |
4dceebe
to
1f5486c
Compare
* WIP: stubs for family pathways * WIP; changes to family pathways * WIP; most questions for family pathways * WIP * Non-HMIS Pathways * Expose first date homeless on pathways client view * Update Pathways and Families Pathways assessments to collect first date homeless * Update tie breaker calculation for pathways assessments * WIP; Family PSH prioritization scheme * Tests for Family PSH prioritization * New prioritization schemes
@dtgreiner we're ready to merge this, but it could use another set of eyes as a bunch has changed since you last looked. |
}, | ||
psh_required: { | ||
title: 'In need of Permanent Supportive Housing', | ||
type: 'String', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do we need description: nil
here too?
} | ||
{}.tap do |options| | ||
options[pathways_assessment_type] = { title: pathways_title, description: pathways_description } | ||
options[family_pathways_assessment_type] = { title: family_pathways_title, description: family_pathways_description } unless Rails.env.production? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Was unless Rails.env.production?
left over in case this went out to production earlier, or is it intentional?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'll add a comment, that's there to allow launching the Transfer assessment prior to the family assessment in production. It will be removed in the near future.
profiles: | ||
- manual |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
wasn't expecting a docker-compose change. I don't see any issues with it, but making sure it was meant to be included here and not in the override
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, this was breaking my local environment. I confirmed with @yakloinsteak, it should not be an issue to include.
Please squash merge this PR
Description
This adds the 2024 version of the non-HMIS client transfer assessment and updates the sync logic for non-HMIS assessments -> clients
Type of change
Checklist before requesting review