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.

Sign up for GitHub

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

Jump to bottom

Producer does not appear in inventory #2799

Closed
OliverUK opened this issue Oct 1, 2018 · 3 comments
Closed

Producer does not appear in inventory #2799

OliverUK opened this issue Oct 1, 2018 · 3 comments
Assignees
@kristinalim
Labels
bug-s3 The bug is stopping a critical or non-critical feature but there is a usable workaround.

Comments

@OliverUK
Copy link
Contributor

OliverUK commented Oct 1, 2018

Description

If I go to the inventory for Stroudco and try and filter for Producer:Wharf Farm, it no longer comes up. They also don't come up if I leave Producer unfiltered and search for their produce. This happens for me logged in as me and it happens when I log in as super admin but it seems to work fine for Nick.

Expected Behavior

Inventory screen loads produce for Wharf Farm

Actual Behavior

Screen displays " No matching products found in your inventory "

Animated Gif/Screenshot

http://recordit.co/k6dAumaPgP

Context

We use the inventory to reset how much a producer has available each week.

Your Environment

Firefox 62.0.2 on Mac 10.12.6
@OliverUK OliverUK added the bug-s3 The bug is stopping a critical or non-critical feature but there is a usable workaround. label Oct 1, 2018
@myriamboure myriamboure mentioned this issue Oct 1, 2018
Closed
@OliverUK
Copy link
Contributor Author

OliverUK commented Oct 7, 2018

With the fix of #2769 this too seems to be working again now. @myriamboure should I close?

@kristinalim kristinalim mentioned this issue Oct 8, 2018
Merged
@myriamboure
Copy link
Contributor

Thanks @OliverUK we will close when we have tested and merged the patch to master, leave it open for now, I have added that issue number to the patch to know we have to close it when tested.

@sigmundpetersen
Copy link
Contributor

I guess we can close this now that #2769 is tested and merged @myriamboure ?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@kristinalim kristinalim

Labels
bug-s3 The bug is stopping a critical or non-critical feature but there is a usable workaround.
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@kristinalim @sigmundpetersen @myriamboure @OliverUK