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

fix: attribute for product without new nutriscore data #10915

Merged
merged 3 commits into from
Oct 23, 2024

Conversation

stephanegigandet
Copy link
Contributor

Some products (in the obsolete collection) do not have new Nutri-Score data (but may have data in an older format). In that case the attribute Nutri-Score should be unknown. This can also happen if we display an older revision of a product.

I'll also recompute the Nutri-Score for the current revision of obsolete products.

Fixes openfoodfacts/smooth-app#5699

@stephanegigandet stephanegigandet enabled auto-merge (squash) October 21, 2024 10:57
@stephanegigandet
Copy link
Contributor Author

"make check_critic" passes locally, no clue why it fails on github...

Copy link

sonarcloud bot commented Oct 21, 2024

@stephanegigandet stephanegigandet merged commit c413719 into main Oct 23, 2024
12 checks passed
@stephanegigandet stephanegigandet deleted the nutriscore-attribute branch October 23, 2024 09:57
stephanegigandet pushed a commit that referenced this pull request Oct 24, 2024
🤖 I have created a release *beep* *boop*
---


##
[2.47.0](v2.46.0...v2.47.0)
(2024-10-24)


### Features

* api docs remove query string from get by barcode and add security to
solve openapi linter errors/warnings
([#10922](#10922))
([b7aefbd](b7aefbd))
* Eco-Score improvements and change of scale: A+ to F
([#10829](#10829))
([87df665](87df665))


### Bug Fixes

* add googleOther bot
([#10912](#10912))
([d56760a](d56760a))
* allow CORS requests for product image move
([#10920](#10920))
([967a47e](967a47e))
* attribute for product without new nutriscore data
([#10915](#10915))
([c413719](c413719))
* cosmetic changes for new Eco-Score grades
([#10921](#10921))
([6e7dbfa](6e7dbfa))
* Revert "fix: remove unused jquery-ui safe-active-element module
([#10708](#10708))"
([#10924](#10924))
([d290c54](d290c54))

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Attributes https://wiki.openfoodfacts.org/Product_Attributes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

ClientException: ClientException: Bad file descriptor, uri=https://world.openfoodfacts.org/cgi/product_jqm2.pl
2 participants