Skip to content

Syncing from upstream OCA/edi-framework (18.0)#166

Merged
bt-admin merged 14 commits into
brain-tec:18.0from
OCA:18.0
Jun 10, 2026
Merged

Syncing from upstream OCA/edi-framework (18.0)#166
bt-admin merged 14 commits into
brain-tec:18.0from
OCA:18.0

Conversation

@bt-admin

Copy link
Copy Markdown

bt_gitbot

duongtq and others added 14 commits June 1, 2026 10:20
Currently translated at 100.0% (12 of 12 strings)

Translation: edi-framework-16.0/edi-framework-16.0-edi_exchange_deduplicate_oca
Translate-URL: https://translation.odoo-community.org/projects/edi-framework-16-0/edi-framework-16-0-edi_exchange_deduplicate_oca/it/
Exchange type action ``act_open_edi_exchange_type_view`` declared ``active_test=False`` in its context to allow displaying archived rules in the ExcType form view.
However, this choice had the side effect of propagating ``active_test=False`` to all relational fields' context, leading to archived records to *always* be displayed.
This fix removes the context key from the action and moves it to the field's definition itself (at model lever, not view level).
Signed-off-by simahawk
Signed-off-by simahawk
@bt-admin bt-admin added the 18.0 label Jun 10, 2026
@bt-admin bt-admin merged commit 4295b6d into brain-tec:18.0 Jun 10, 2026
4 checks passed
@codecov

codecov Bot commented Jun 10, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 96.00000% with 6 lines in your changes missing coverage. Please review.
✅ Project coverage is 94.09%. Comparing base (a16174f) to head (e307b06).
⚠️ Report is 87 commits behind head on 18.0.

Files with missing lines Patch % Lines
edi_exchange_deduplicate_oca/models/edi_backend.py 70.00% 4 Missing and 2 partials ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             18.0     #166      +/-   ##
==========================================
+ Coverage   94.06%   94.09%   +0.03%     
==========================================
  Files         225      233       +8     
  Lines        6516     6658     +142     
  Branches      418      429      +11     
==========================================
+ Hits         6129     6265     +136     
- Misses        280      284       +4     
- Partials      107      109       +2     

☔ View full report in Codecov by Harness.
📢 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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants