search

OpenHousing / hhs

0 stars 2 forks source link

Only clients from Richmond CoC should display in application. #16

Open vykster opened 7 years ago

vykster commented 7 years ago

Need details here on how best to implement this feature.

Any instance of the tool should only display records from specific CoCs, even if the source data includes more that the ones specified. Only BrightHive staff should be able to change what CoCs are included in any particular deployment of the application.

Also need field/ value for Richmond, VA to implement this.

vykster commented 6 years ago

@tplagge @thethingstheycoded I don't have Richmond's CoC ID documented anywhere. Could you share that with me and kevin?

tplagge commented 6 years ago

In my data pull from Margot, the CoC "code" for Richmond is "Richmond CoC". Based on this file: https://www.hudexchange.info/resources/documents/FY-2017-CoC-Geographic-Codes.pdf it might show up as 511308.

On Fri, Dec 8, 2017 at 3:06 PM, Vyki Englert notifications@github.com wrote:

@tplagge https://github.com/tplagge @thethingstheycoded https://github.com/thethingstheycoded I don't have Richmond's CoC ID documented anywhere. Could you share that with me and kevin?

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/OpenHousing/hhs/issues/16#issuecomment-350371769, or mute the thread https://github.com/notifications/unsubscribe-auth/ABpxdi87LLjySjcwm-W3hZfS186-EphVks5s-aTagaJpZM4QW4I- .

KClough commented 6 years ago

@tplagge do you happen to know where I should be getting the client's CoC ID? I didn't see anything that stood out in the /clients or /enrollments response from HMISLynk.

KClough commented 6 years ago

Note, CoC ID should be in EnrollmentCoC APIs. https://hmis-api.github.io/2017-hmis-api/#clients__clientid__enrollments__enrollmentid__enrollmentcocs_get - from Eric Jahn

KClough commented 6 years ago

Note, Looks like that url is giving a 404 response. Waiting on update from Eric.

KClough commented 6 years ago

@vykster + @tplagge

If I load a client with multiple enrollments, but only some enrollments have the specified cocCode, should I ignore those enrollments?

tplagge commented 6 years ago

Yes, that's the right approach.

On Thu, Dec 14, 2017 at 4:04 PM, Kevin Clough notifications@github.com wrote:

@vykster https://github.com/vykster + @tplagge https://github.com/tplagge

If I load a client with multiple enrollments, but only some enrollments have the specified cocCode, should I ignore those enrollments?

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/OpenHousing/hhs/issues/16#issuecomment-351850175, or mute the thread https://github.com/notifications/unsubscribe-auth/ABpxdj5iAQea4Kwpnok6q0bx3Huz9X4kks5tAZuIgaJpZM4QW4I- .

KClough commented 6 years ago

@vykster + @tplagge

I'm loading Coc_IDs from HMISLynk. The majority of them so far are "VA-521". I haven't seen any that are 6 digit numeric like the suggested "511308".

Could you verify this is expected?

tplagge commented 6 years ago

VA-521 is the Balance of State, ie rural Virginia. Are there many VA-500s? That's Richmond.

On Fri, Dec 15, 2017, 4:09 PM Kevin Clough notifications@github.com wrote:

@vykster https://github.com/vykster + @tplagge https://github.com/tplagge

I'm loading Coc_IDs from HMISLynk. The majority of them so far are "VA-521". I haven't seen any that are 6 digit numeric like the suggested "511308".

Could you verify this is expected?

— You are receiving this because you were mentioned.

Reply to this email directly, view it on GitHub https://github.com/OpenHousing/hhs/issues/16#issuecomment-352124471, or mute the thread https://github.com/notifications/unsubscribe-auth/ABpxdn0e6j42O4ZvJF5V2MItQyv-mKsjks5tAu4sgaJpZM4QW4I- .

KClough commented 6 years ago

@tplagge I'm just spot checking at this point. I added some test code to store the CoCID for each client. I will follow up once I've had a chance to review a larger set of data.