Skip to content

Syncing from upstream OCA/knowledge (18.0)#662

Merged
bt-admin merged 3 commits intobrain-tec:18.0from
OCA:18.0
Mar 4, 2026
Merged

Syncing from upstream OCA/knowledge (18.0)#662
bt-admin merged 3 commits intobrain-tec:18.0from
OCA:18.0

Conversation

@bt-admin
Copy link

@bt-admin bt-admin commented Mar 4, 2026

bt_gitbot

victoralmau and others added 3 commits March 3, 2026 12:03
…ry and send mail

Example use case:
- Modify the Odoo Features category, check Require approval, and set Approver group > Administration / Access Rights
- Modify the Marc Demo user and define Document Knowledge > Editor permissions
- Go to the Odoo Features category and, taking into account the ID, delete the mail_message record table with the same ID from the database
- With the Marc Demo user, go to the Knowledge > Odoo Features menu and enter the Odoo 15.0 Functional Demo page (it is important to follow these specific steps)
- Modify the page and save

Before this change, an error could occur because the action https://github.com/OCA/knowledge/blob/0881d75a35e7fc2f814110df4878177e805e589b/document_page/views/document_page_category. xml#L83C17-L83C42 defined a default_parent_id=x (page category ID) if there was no mail_message linked to that category ID; now the value of default_parent_id is removed to avoid this specific error.

TT61290
Signed-off-by pedrobaeza
@bt-admin bt-admin added the 18.0 label Mar 4, 2026
@bt-admin bt-admin merged commit b2a4499 into brain-tec:18.0 Mar 4, 2026
4 checks passed
@codecov
Copy link

codecov bot commented Mar 4, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 95.79%. Comparing base (0881d75) to head (f747aa3).
⚠️ Report is 33 commits behind head on 18.0.

Additional details and impacted files
@@            Coverage Diff             @@
##             18.0     #662      +/-   ##
==========================================
+ Coverage   95.77%   95.79%   +0.02%     
==========================================
  Files          82       82              
  Lines        1088     1095       +7     
  Branches       83       83              
==========================================
+ Hits         1042     1049       +7     
  Misses         31       31              
  Partials       15       15              

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

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.

3 participants