Skip to content

[18.0][FIX+IMP] connector_extension: post-migration fixes and improvements#891

Open
deeniiz wants to merge 3 commits into18.0from
18.0-fix-imp-connector_extension
Open

[18.0][FIX+IMP] connector_extension: post-migration fixes and improvements#891
deeniiz wants to merge 3 commits into18.0from
18.0-fix-imp-connector_extension

Conversation

@deeniiz
Copy link
Copy Markdown
Collaborator

@deeniiz deeniiz commented Apr 15, 2026

No description provided.

deeniiz added 3 commits April 15, 2026 12:34
In Odoo 18, queue_job forbids commits during job execution via
_prevent_commit to avoid releasing the job lock prematurely.
The cr.commit() in bind_export was an optimization for concurrent
binding creation that is no longer compatible. The _retry_unique_violation
context manager still handles race conditions via PostgreSQL's UNIQUE
constraint without needing an explicit commit.
@codecov
Copy link
Copy Markdown

codecov Bot commented Apr 15, 2026

Codecov Report

❌ Patch coverage is 0% with 3 lines in your changes missing coverage. Please review.
✅ Project coverage is 29.80%. Comparing base (e028182) to head (c6b6c95).

Files with missing lines Patch % Lines
connector_extension/components/binder.py 0.00% 3 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             18.0     #891      +/-   ##
==========================================
+ Coverage   29.76%   29.80%   +0.03%     
==========================================
  Files          17       17              
  Lines        1233     1228       -5     
  Branches      252      251       -1     
==========================================
- Hits          367      366       -1     
+ Misses        866      862       -4     

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@deeniiz deeniiz requested a review from eantones April 15, 2026 10:43
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.

1 participant