Explore disabling InventoryItem about human-essentials HOT 9 CLOSED

Started looking at this... there really isn't a way to do this halfway and validate it. A whole crapload of specs fail because they're testing very specific methods or items that won't exist. Basically the end result of this would be a branch where all reference to inventory items and any method that deals with them are completely expunged.

One thing that tripped me up is that a bunch of specs do storage_location.items, which goes through inventory_items and which fail a lot. :(

from human-essentials.

Hmm. I'm not sure that I've throughly tested the inventory history feature with event sourcing. I expect we'll need to have it only go back to the snapshot date, yes?

from human-essentials.

Yep. We can only show history via events.

from human-essentials.

This issue is marked as stale due to no activity within 30 days. If no further activity is detected within 7 days, it will be unassigned.

from human-essentials.

I believe the status on this is that we have determined that we will need the inventoryitem until such time as we don't care about the inventory history before event sourcing. This is estimated to be (hopefully) January 1 2026. Especially as the pre-event sourcing inventory history is suspect, we should be able to get stakeholder buy-in for that.

from human-essentials.

We will need InventoryItem, the model, but we should be removing all code that creates, updates or reads them.

from human-essentials.

Except for whatever is necessary to use the versions.

from human-essentials.

Yup. Just that one place.

from human-essentials.

I'm adding a "Waiting for" to the HE Project, and putting in an entry about this dated Jan 1 2026. Closing it as a current issue.

from human-essentials.