Skip to content
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

Include period supplies in kits in NDBN report values that are based on period supplies #4283

Open
2 tasks
cielf opened this issue Apr 17, 2024 · 5 comments
Open
2 tasks
Assignees

Comments

@cielf
Copy link
Collaborator

cielf commented Apr 17, 2024

Summary

Period supply values in the Annual Survey should include period supplies in kits

Why?

Better reporting of actual distribution

Details

Similar to issue #3939... You may need to use SQL to extract the information
Look at services/reports/period_supply_report_service
Assume 1 kit per individual

The following values in the report will need attention:
Period Supplies distributed
Period Supplies per adult per month [Note: this should be per individual per month -- confirmed with NDBN liason]
% period supplies donated -- note that most banks make their own kits, but we do have some who have kits donated -- so this is a slightly different twist from #3939
% period supplies bought -- afaik, we don't have any cases of kits being bought, but we don't prevent it.

Criteria for completion

  • values in report regarding period supplies reflect period supplies in kits
  • tesets demonstrating the changes
@cielf
Copy link
Collaborator Author

cielf commented Apr 17, 2024

This is going to be assigned to @jadekstewart3. This comments just here so I can do that. Apparently I actually need Jade to comment on this issue to do that!

@jadekstewart3
Copy link
Contributor

I can take this! Thank you :)

@cielf
Copy link
Collaborator Author

cielf commented Apr 18, 2024

Thank you!

@jadekstewart3
Copy link
Contributor

Just an update: Still working on this. I was OOO the last week an a half, so I am hoping to get some more done this week and next :)

@cielf
Copy link
Collaborator Author

cielf commented May 27, 2024

Hi @jadekstewart3 -- just a heads up that I'm doing some rework of the whole what-goes-in-what-bucket due to conversations with NDBN, and things I've found that were just not right! keep an eye on PR # 4361.
There is going to be a rework of what is in period_supplies -- making it depend on the category rather than the partner keys --- it turns out the initialization of the seed and what is in prod had different categories, which explains why someone coded it with the partner keys! The latest on that should (I hope) be up later today, though not merged.

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

No branches or pull requests

2 participants