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

Ingredients analysis "We need your help!" message always displayed even when there are no unrecognized ingredients #6341

Closed
stephanegigandet opened this issue Jan 25, 2022 · 0 comments · Fixed by #6342 or #6232
Closed

Ingredients analysis "We need your help!" message always displayed even when there are no unrecognized ingredients #6341

stephanegigandet opened this issue Jan 25, 2022 · 0 comments · Fixed by #6342 or #6232
Assignees
@stephanegigandet
Labels
🐛 bug This is a bug, not a feature request. 🥗🔍 Ingredients analysis https://wiki.openfoodfacts.org/Ingredients_Extraction_and_Analysis

Comments

@stephanegigandet
Copy link
Contributor

Describe the bug

Introduced when adding the specific ingredients parsing feature.

To Reproduce

https://world.openfoodfacts.org/cgi/test_ingredients_analysis.pl?ingredients_text=fresh+potato&type=add&action=process

Expected behavior

Message shown only when there are some unrecognized ingredients.

Screenshots

No response

Additional context

No response

Type of device

Browser

Browser version

No response

Number of products impacted

No response

Time per product

No response
@stephanegigandet stephanegigandet added 🐛 bug This is a bug, not a feature request. 🥗🔍 Ingredients analysis https://wiki.openfoodfacts.org/Ingredients_Extraction_and_Analysis labels Jan 25, 2022
@stephanegigandet stephanegigandet self-assigned this Jan 25, 2022
stephanegigandet added a commit that referenced this issue Jan 25, 2022
@stephanegigandet
fix: show 'we need your help' message for ingredients analysis only w…
f1c2f67 
…hen needed - fixes #6341
@stephanegigandet stephanegigandet mentioned this issue Jan 25, 2022
Merged
stephanegigandet added a commit that referenced this issue Jan 26, 2022
@stephanegigandet
fix: show 'we need your help' message for ingredients analysis only w…
9e001c1 
…hen needed - fixes #6341 (#6342)
@github-actions github-actions bot mentioned this issue Jan 26, 2022
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@stephanegigandet stephanegigandet

Labels
🐛 bug This is a bug, not a feature request. 🥗🔍 Ingredients analysis https://wiki.openfoodfacts.org/Ingredients_Extraction_and_Analysis
Projects
None yet
Milestone
No milestone
1 participant
@stephanegigandet