tulip / ppe-logistics

Issue Tracking for PPELogistics.org
2 stars 1 forks source link

Supplier Demands - After I adjusted inventory incorrectly. I just typed in an amount without picking a product item. I get this listed in my inventory. See BUG05022020f.JPG #125

Open jackkrooss opened 4 years ago

jackkrooss commented 4 years ago

https://drive.google.com/open?id=1QUkwQPypD49TSv1_uu6CLYER_CDHSYui

jackkrooss commented 4 years ago

BUG20200502f

ianslai commented 4 years ago

I wonder if this is related to #167 ? I tried to reproduce this issue without success; I'm getting an error saying "you must select an item to add first" which is preventing me from adding a row with no part number. In #167 I fixed the issue where you can select "Adjust Inventory" in the Inventory Manager without specifying a site or item.

jackkrooss commented 4 years ago

Likely the other fix fixed it, but it is always dicey to declare something closed when you can't replicate it. Is it possible to back out your fix of

167 and see if you can generate the error?

On Tue, May 19, 2020, 5:24 PM ianslai notifications@github.com wrote:

I wonder if this is related to #167 https://github.com/tulip/ppe-logistics/issues/167 ? I tried to reproduce this issue without success; I'm getting an error saying "you must select an item to add first" which is preventing me from adding a row with no part number. In #167 https://github.com/tulip/ppe-logistics/issues/167 I fixed the issue where you can select "Adjust Inventory" in the Inventory Manager without specifying a site or item.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/tulip/ppe-logistics/issues/125#issuecomment-631161427, or unsubscribe https://github.com/notifications/unsubscribe-auth/APJASPKLOWEFCKWM35YVKODRSMPLTANCNFSM4M4ICCQA .

ianslai commented 4 years ago

I briefly tried disabling my fix from #167 but wasn't able to reproduce the problem.

jackkrooss commented 4 years ago

I will try to reproduce problem tomorrow and be very explicit if I can break it.

On Tue, May 19, 2020, 6:36 PM ianslai notifications@github.com wrote:

I briefly tried disabling my fix from #167 https://github.com/tulip/ppe-logistics/issues/167 but wasn't able to reproduce the problem.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/tulip/ppe-logistics/issues/125#issuecomment-631181454, or unsubscribe https://github.com/notifications/unsubscribe-auth/APJASPMKJJQV7M3GRXKKAK3RSMX2HANCNFSM4M4ICCQA .