-
-
Notifications
You must be signed in to change notification settings - Fork 393
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
feat: add explanation with category in the Nutri-Score and Eco-Score knowledge panels when not applicable #6844
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM but you forgot an image. Also I was expecting a new nutriscore-not-applicable logo ;-)
Also, @stephanegigandet, could be good to add a test for eco-score not applicable… |
@alexgarel for the logos, I made a separate PR: Santé publique France has an official logo for "Nutri-Score unknown", so I think we should use it. I think we can use the same logo for "Nutri-Score not-applicable" as well. Same for Eco-Score unknown and not-applicable (yet), we can use the same logo. |
Co-authored-by: Alex Garel <[email protected]>
…rver into nutri-eco
Good point, I added a test. |
Kudos, SonarCloud Quality Gate passed! 0 Bugs No Coverage information |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good.
When a product does not have a Nutri-Score or an Eco-Score because the product category is excluded from the Nutri-Score and Eco-Score, we now explain why in the corresponding knowledge panel, and we list the category.
Fixes #6092 and #6774