human-essentials icon indicating copy to clipboard operation
human-essentials copied to clipboard

[BUG]: cannot update donations when inventory is below distribution

Open jimmyli97 opened this issue 1 year ago • 3 comments

Is there an existing issue for this?

  • [X] I have searched the existing issues

Current Behavior

Related to #4172

After creating a distribution which decreases the inventory for an item, attempts to update donations which have that item fail with "Error updating donation: Requested items exceed the available inventory", even if the quantity for that item remains the same or increases.

Expected Behavior

It should let you update the donation if the quantity for that item remains the same or increases.

Steps To Reproduce

  1. bin/setup followed by bin/start
  2. In browser, create a distribution at a storage location which decreases the inventory for an item
  3. In browser, attempt to update a donation which contains that item at that storage location

Environment

main commit 524eaa

Criteria for Completion

[] fix [] write tests

Anything else?

No response

Code of Conduct

  • [X] I've read the Code of Conduct and understand my responsibilities as a member of the Ruby for Good community

jimmyli97 avatar Jul 11 '24 17:07 jimmyli97

Hey @jimmyli97 If this is what I think it is, it's taken care of by our recent inventory-related changes which are in beta at the moment. If you enable the flipper flag 'read_events", it should take care of it.

cielf avatar Jul 11 '24 17:07 cielf

Hey @jimmyli97 If this is what I think it is, it's taken care of by our recent inventory-related changes which are in beta at the moment. If you enable the flipper flag 'read_events", it should take care of it.

@cielf yes that fixed it. to turn on that flag I've been adding the flag in seeds.rb, is there another way? I could make a commit to add that and close this issue.

jimmyli97 avatar Jul 11 '24 18:07 jimmyli97

We're keeping the seed to be what the majority of banks have for ease of support. The way you enable the flag to go to localhost:3000/flipper, userid admin, password password, click on "Add feature", add read_events, click "add_feature", click "fully enable".

cielf avatar Jul 11 '24 20:07 cielf

Looks like this issue has been completed. Could it be closed?

coalest avatar Nov 12 '24 18:11 coalest

Yup, thanks.

cielf avatar Nov 12 '24 20:11 cielf