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

feat: sync number of product public/pro in CRM #10684

Merged

Conversation

4nt0ineB
Copy link
Member

What

The daily script (save_org_product_data_daily_off_pro.pl) which updates orgs data now sync the number of public/prod products with CRM.

@4nt0ineB 4nt0ineB requested a review from a team as a code owner August 12, 2024 13:20
@github-actions github-actions bot added 🏭 Producers Platform https://wiki.openfoodfacts.org/Platform_for_producers 🏭 Producers Platform - Odoo OdooCRM is used by the Open Food Facts team to manage producers (and more) relationships 🏭 Orgs labels Aug 12, 2024
Copy link

@codecov-commenter
Copy link

Codecov Report

Attention: Patch coverage is 31.42857% with 24 lines in your changes missing coverage. Please review.

Project coverage is 49.37%. Comparing base (dc04d18) to head (20f6a29).
Report is 563 commits behind head on main.

Files Patch % Lines
lib/ProductOpener/Orgs.pm 39.13% 13 Missing and 1 partial ⚠️
lib/ProductOpener/CRM.pm 16.66% 10 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main   #10684      +/-   ##
==========================================
- Coverage   49.54%   49.37%   -0.18%     
==========================================
  Files          67       77      +10     
  Lines       20650    22005    +1355     
  Branches     4980     5252     +272     
==========================================
+ Hits        10231    10864     +633     
- Misses       9131     9815     +684     
- Partials     1288     1326      +38     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Copy link
Contributor

@stephanegigandet stephanegigandet left a comment

Choose a reason for hiding this comment

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

Thank you!

@stephanegigandet stephanegigandet merged commit 026ff6a into openfoodfacts:main Aug 13, 2024
12 checks passed
stephanegigandet pushed a commit that referenced this pull request Aug 13, 2024
🤖 I have created a release *beep* *boop*
---


##
[2.41.0](v2.40.0...v2.41.0)
(2024-08-13)


### Features

* generate historical events from product revisions
([#10523](#10523))
([42508e1](42508e1))
* sync number of product public/pro in CRM
([#10684](#10684))
([026ff6a](026ff6a))


### Bug Fixes

* 3 fixes to make /products/[code1],[code2] work again
([#10669](#10669))
([7fc07fc](7fc07fc))
* Automatically unselect images when source image is deleted
([#10692](#10692))
([588897a](588897a))
* **docs:** correct OpenAPI definition by fixing missing reference
([#10659](#10659))
([4f0f10e](4f0f10e))
* gen_users_emails.pl
([#10686](#10686))
([00e4c26](00e4c26))
* Show missing nutrients in Nutri-Score panel
([#10655](#10655))
([a1b11eb](a1b11eb))
* UTM mobile
([#10665](#10665))
([965041d](965041d))

---
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
🏭 Orgs 🏭 Producers Platform - Odoo OdooCRM is used by the Open Food Facts team to manage producers (and more) relationships 🏭 Producers Platform https://wiki.openfoodfacts.org/Platform_for_producers
Projects
Status: Done
Status: Done
Development

Successfully merging this pull request may close these issues.

3 participants