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

Confusing variants count in OC #5001

Open
RachL opened this issue Mar 16, 2020 · 6 comments
Open

Confusing variants count in OC #5001

RachL opened this issue Mar 16, 2020 · 6 comments
Labels
bug-s3 The bug is stopping a critical or non-critical feature but there is a usable workaround. feedback-needed

Comments

@RachL
Copy link
Contributor

RachL commented Mar 16, 2020

Description

We have recently improved our count of variants, but there still are some weird cases.

Here is an example with french prod shop 173 :

  1. The list of OC shows 65 variants:

image

  1. The sum in incoming is correct.

  2. In outgoing it says that I have selected 64 variants. When I unfold the list it says that it is showing me 65 out of 64 variants...

image

Steps to Reproduce

  1. Go to /admin/order_cycles on French production
  2. Have a look at OC 3063 for shop 173

Severity

bug-s3: a feature is broken but there is a workaround

Your Environment

  • Version used: v2.7.8
  • Browser name and version: Firefox 73
  • Operating System and version (desktop or mobile): Desktop
@RachL RachL added the bug-s3 The bug is stopping a critical or non-critical feature but there is a usable workaround. label Mar 16, 2020
@RachL
Copy link
Contributor Author

RachL commented Mar 17, 2020

Ok, because I'm a big cheater, I will use this one as an epic to group the issues on this currently in dev ready.

@RachL RachL added the epic Group of issues label Mar 17, 2020
@luisramos0
Copy link
Contributor

The problem described in this issue is not seen here any longer:
https://openfoodfrance.org/admin/order_cycles/3063/outgoing

Anyway, this epic's description looks like a duplicate of #4806 (one of the issues inside it).

@luisramos0
Copy link
Contributor

@RachL fyi I have just fixed this problem
image

here in transifex (missing space):
https://www.transifex.com/open-food-foundation/open-food-network/translate/#fr/enyml/194011289?q=key%3Avariants_loaded

@RachL
Copy link
Contributor Author

RachL commented May 26, 2020

Anyway, this epic's description looks like a duplicate of #4806 (one of the issues inside it).

Yes I cheated a bit when creating the epic and forgot to update the description 😁

@luisramos0
Copy link
Contributor

Adding feedback needed to make it clear this is blocked by the pending decision in #4806

@audez
Copy link
Collaborator

audez commented Aug 11, 2022

Hi! We have a hub manager that raised the problem (the fact that in step 3 the numbers didn't make sense, ex: 4 of 0 Variants Loaded)

@sigmundpetersen sigmundpetersen removed the epic Group of issues label Feb 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug-s3 The bug is stopping a critical or non-critical feature but there is a usable workaround. feedback-needed
Projects
Status: All the things 💤
Development

No branches or pull requests

4 participants