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

Adding reporting categories to items (and base items) #5007

Open
3 tasks
cielf opened this issue Feb 10, 2025 · 4 comments
Open
3 tasks

Adding reporting categories to items (and base items) #5007

cielf opened this issue Feb 10, 2025 · 4 comments
Assignees

Comments

@cielf
Copy link
Collaborator

cielf commented Feb 10, 2025

Summary

Rework annual reporting so that it is based on properties that are directly on the items, rather than looking at the base items

Why

Details

This will involve adding a new "NDBN reporting category" concept to the items
This will include disposable diapers; cloth diapers; adult incontinence; period underwear; pads; period liners; tampons; period reusables; period other.

We will add the reporting categories to the base items and populate it according to rules followed in the annual report. (they are complicated). Note that period items are going to be split out further than they currently are.

Then we copy the reporting categories from the base items to the current items.

When new banks are created, the new items will have the reporting categories from the corresponding base item.

Annual reports will need to be changed to use the new reporting categories. This is also a significant chunk of work.

We should change "filter by base item" to "filter by reporting category"

This field should not be populated / editable for items that are representing kits. (they have base item kit, but there are other ways to tell and we should rely on those to check this.) It should be mandatory otherwise, though.

Implement this is a way that it would be relatively easy to add in new reporting categories -- we don't expect to have changes more than once every couple years. A simple crud interface for superadmins would be ideal or we can just do a data migration as needed.

Once this is all done, we will be able to remove the direct link from base item to item and make the base items merely the seed.

Criteria for completion

  • behaviour as described
  • tests to support the above behaviour
  • update user guide to support the above behaviour

Suggested breakout

Step 1

--- add fields to base item
---- we could do the population of the base items 'by hand' instead of algorithmically. That makes a lot of sense. (CL can provide the mapping)

Step 2

--- add fields to item.
--- migration of the current from base items to items, and reporting categories on population of items for new banks from the base items

Step 3

--- make the changes to the annual report
-- this will need to be fleshed out a bit. (CL to flesh out)

Step 4

--- allow viewing/editing of the report category on items.
--- add filtering by report category where we now have filtering by base item
--- remove filtering by base item

@coalest
Copy link
Collaborator

coalest commented Feb 20, 2025

Will users be able to set the reporting_category for an item to be value they want? Or will we limit the options to only those mentioned (Adult Incontinence, Cloth diapers, etc)?

@cielf
Copy link
Collaborator Author

cielf commented Feb 20, 2025

Only those mentioned. It is possible that we may add reporting categories in the future, but this is essentially something so they can say how it gets counted in the Annual Survey.

@coalest
Copy link
Collaborator

coalest commented Feb 24, 2025

@cielf For the reporting categories, I noticed that most of the values are title case ("Cloth Diapers" and "Adult Incontinence"), except for one: "Disposable diapers" (lower case d in diapers).

Is it okay if I make it "Disposable Diapers" instead? That will make it easier for me to humanize the categories.

@cielf
Copy link
Collaborator Author

cielf commented Feb 25, 2025

I'm fine with that -- it was almost certainly a slip of the fingers.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants