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

[14.0][FIX] connector_oxigesti: removing items from Odoo did not mark the corresponding records in Oxigesti as deprecated #352

Closed
wants to merge 8 commits into from

Conversation

FrankC013
Copy link
Contributor

No description provided.

@FrankC013 FrankC013 changed the title [14.0][IMP] connector_oxigesti: logic for deprecated field of pricelist item [WIP][14.0][IMP] connector_oxigesti: logic for deprecated field of pricelist item Jan 2, 2024
Copy link

codecov bot commented Jan 2, 2024

Codecov Report

Attention: Patch coverage is 38.09524% with 182 lines in your changes missing coverage. Please review.

Project coverage is 44.95%. Comparing base (5dd2c64) to head (e50a5c0).
Report is 32 commits behind head on 14.0.

Current head e50a5c0 differs from pull request most recent head 21bd969

Please upload reports for the commit 21bd969 to get more accurate results.

Files Patch % Lines
connector_oxigesti/common/tools.py 11.39% 70 Missing ⚠️
...nnector_oxigesti/models/oxigesti_binding/common.py 19.04% 34 Missing ⚠️
..._oxigesti/models/product_pricelist_item/binding.py 44.73% 19 Missing and 2 partials ⚠️
connector_oxigesti/components/deleter.py 67.64% 11 Missing ⚠️
connector_oxigesti/components/adapter.py 9.09% 10 Missing ⚠️
...oxigesti/models/product_pricelist_item/exporter.py 38.46% 8 Missing ⚠️
...ctor_oxigesti/models/product_pricelist/listener.py 66.66% 3 Missing ⚠️
...oxigesti/models/product_pricelist_item/listener.py 66.66% 3 Missing ⚠️
...ector_oxigesti/models/product_template/listener.py 40.00% 3 Missing ⚠️
connector_oxigesti/models/res_partner/listener.py 66.66% 3 Missing ⚠️
... and 9 more
Additional details and impacted files
@@            Coverage Diff             @@
##             14.0     #352      +/-   ##
==========================================
- Coverage   44.98%   44.95%   -0.03%     
==========================================
  Files         747      751       +4     
  Lines       12972    13145     +173     
  Branches     2836     2867      +31     
==========================================
+ Hits         5835     5909      +74     
- Misses       7005     7102      +97     
- Partials      132      134       +2     

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

@FrankC013 FrankC013 force-pushed the 14.0-imp-connector_oxigesti-deprecated branch from e61bd5e to 738b953 Compare January 5, 2024 08:25
@FrankC013 FrankC013 requested a review from eantones January 5, 2024 08:25
@FrankC013 FrankC013 force-pushed the 14.0-imp-connector_oxigesti-deprecated branch from 9d0c14a to 0965df8 Compare January 5, 2024 16:40
@eantones eantones changed the title [WIP][14.0][IMP] connector_oxigesti: logic for deprecated field of pricelist item [14.0][IMP] connector_oxigesti: logic for deprecated field of pricelist item Jan 9, 2024
@FrankC013 FrankC013 requested a review from eantones January 16, 2024 17:08
@FrankC013 FrankC013 force-pushed the 14.0-imp-connector_oxigesti-deprecated branch 2 times, most recently from 6466ceb to 953f37d Compare January 17, 2024 13:13
@FrankC013 FrankC013 force-pushed the 14.0-imp-connector_oxigesti-deprecated branch from 953f37d to b7492a0 Compare January 18, 2024 14:44
@FrankC013 FrankC013 requested a review from eantones January 18, 2024 14:45
@FrankC013 FrankC013 force-pushed the 14.0-imp-connector_oxigesti-deprecated branch 2 times, most recently from b813b9c to 2edf70c Compare January 18, 2024 14:52
@FrankC013 FrankC013 force-pushed the 14.0-imp-connector_oxigesti-deprecated branch from 2edf70c to 5b937f6 Compare January 19, 2024 15:58
@FrankC013 FrankC013 force-pushed the 14.0-imp-connector_oxigesti-deprecated branch 6 times, most recently from eb26f4c to 89bbcf0 Compare February 6, 2024 09:24
connector_oxigesti/components/adapter.py Outdated Show resolved Hide resolved
connector_oxigesti/components/deleter.py Outdated Show resolved Hide resolved
connector_oxigesti/models/oxigesti_binding/common.py Outdated Show resolved Hide resolved
connector_oxigesti/models/product_pricelist/listener.py Outdated Show resolved Hide resolved
connector_oxigesti/models/product_template/listener.py Outdated Show resolved Hide resolved
connector_oxigesti/models/oxigesti_binding/common.py Outdated Show resolved Hide resolved
connector_oxigesti/models/product_template/listener.py Outdated Show resolved Hide resolved
connector_oxigesti/models/res_partner/listener.py Outdated Show resolved Hide resolved
@eantones eantones force-pushed the 14.0-imp-connector_oxigesti-deprecated branch from 0e5e9c1 to 21bd969 Compare February 27, 2024 19:11
Copy link

There hasn't been any activity on this pull request in the past 4 months, so it has been marked as stale and it will be closed automatically if no further activity occurs in the next 30 days.
If you want this PR to never become stale, please ask a PSC member to apply the "no stale" label.

@github-actions github-actions bot added the stale label Jun 30, 2024
@eantones eantones removed the stale label Jun 30, 2024
Copy link

github-actions bot commented Nov 3, 2024

There hasn't been any activity on this pull request in the past 4 months, so it has been marked as stale and it will be closed automatically if no further activity occurs in the next 30 days.
If you want this PR to never become stale, please ask a PSC member to apply the "no stale" label.

@eantones
Copy link
Member

eantones commented Dec 5, 2024

Superseded by #443

@eantones eantones closed this Dec 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants