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

Only show primary producers on shopfront list of producers (fix #4218) #7015

Merged

Conversation

andrewpbrett
Copy link
Contributor

What? Why?

Closes #4218

This limits the list of producers on the shopfront Producers tab to those enterprises who are primary producers; that is, it does not show hub-only enterprises.

What should we test?

Enterprises that are hubs and not primary producers should not show up on that tab for a hub.

Release notes

Fixed a bug where non-producing enterprises would show up on the shopfront's list of producers.

Changelog Category: User facing changes

Dependencies

Documentation updates

Copy link
Contributor

@sauloperez sauloperez left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm missing at least a unit test (an additional integration test would also be helpful though) to ensure we don't break it in the future.

@andrewpbrett andrewpbrett requested a review from sauloperez March 8, 2021 17:11
@filipefurtad0 filipefurtad0 self-assigned this Mar 17, 2021
@filipefurtad0 filipefurtad0 added the pr-staged-uk staging.openfoodnetwork.org.uk label Mar 17, 2021
@filipefurtad0
Copy link
Contributor

Hey @andrewpbrett ,

Before this PR:

image

The Hacienda and Hacienda Distributors are not producers (both are hubs); After this PR these are gone:

image

Cool, thanks for this fix! Good to go 👍

@filipefurtad0 filipefurtad0 removed the pr-staged-uk staging.openfoodnetwork.org.uk label Mar 17, 2021
@Matt-Yorkley Matt-Yorkley merged commit 751b934 into openfoodfoundation:master Mar 17, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

A shop producer tab is showing hubs as well
4 participants