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: fix crm sync #10747

Merged
merged 5 commits into from
Sep 6, 2024
Merged

fix: fix crm sync #10747

merged 5 commits into from
Sep 6, 2024

Conversation

4nt0ineB
Copy link
Member

What

  • the fixes on the migration script we added directly (in order for the main to be up to date)
  • various org contact info that were not synced

@4nt0ineB 4nt0ineB requested a review from a team as a code owner August 28, 2024 14:46
@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 28, 2024
@codecov-commenter
Copy link

codecov-commenter commented Aug 28, 2024

Codecov Report

Attention: Patch coverage is 20.00000% with 4 lines in your changes missing coverage. Please review.

Project coverage is 49.23%. Comparing base (dc04d18) to head (88c7c65).
Report is 629 commits behind head on main.

Files with missing lines Patch % Lines
lib/ProductOpener/CRM.pm 0.00% 3 Missing ⚠️
lib/ProductOpener/Orgs.pm 50.00% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main   #10747      +/-   ##
==========================================
- Coverage   49.54%   49.23%   -0.32%     
==========================================
  Files          67       77      +10     
  Lines       20650    22087    +1437     
  Branches     4980     5274     +294     
==========================================
+ Hits        10231    10874     +643     
- Misses       9131     9887     +756     
- 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

@alexgarel alexgarel enabled auto-merge (squash) September 6, 2024 15:47
Copy link

sonarcloud bot commented Sep 6, 2024

@alexgarel alexgarel merged commit 336647f into openfoodfacts:main Sep 6, 2024
12 checks passed
stephanegigandet pushed a commit that referenced this pull request Sep 11, 2024
🤖 I have created a release *beep* *boop*
---


##
[2.44.0](v2.43.0...v2.44.0)
(2024-09-09)


### Features

* 99 French "indice de durabilité" Label images
([#10734](#10734))
([be46cf8](be46cf8))
* 99 indice de reparabilite logos
([#10748](#10748))
([e2c302b](e2c302b))
* new [taxonomy].extended.json with extended synonyms
([#10744](#10744))
([e1a485e](e1a485e))
* Open folksonomies for all connected users
([#10771](#10771))
([160b792](160b792))
* prepare folksonomy for multi-domain
([#10775](#10775))
([6f8e772](6f8e772))
* script to extract historical data for a field (e.g. Nutri-Score) for a
set of products
([#10408](#10408))
([d40bcf6](d40bcf6))


### Bug Fixes

* block more AI bots
([#10754](#10754))
([852ca5f](852ca5f))
* fix crm sync
([#10747](#10747))
([336647f](336647f))
* import_prod_data not working
([#10750](#10750))
([a895cd3](a895cd3)),
closes
[#10749](#10749)
* remove non-JS list of products
([#10743](#10743))
([6805b9f](6805b9f))

---
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.

4 participants