Skip to content

[19.0][MIG] document_knowledge: Migration to 19.0#590

Closed
gregory-rutecki wants to merge 55 commits intoOCA:19.0from
gregory-rutecki:19.0-mig-document_knowledge
Closed

[19.0][MIG] document_knowledge: Migration to 19.0#590
gregory-rutecki wants to merge 55 commits intoOCA:19.0from
gregory-rutecki:19.0-mig-document_knowledge

Conversation

@gregory-rutecki
Copy link

Summary

Complete migration of document_knowledge to Odoo 19.0.

Changes:

  • Version bump 18.0 → 19.0
  • ir.module.category → added res.groups.privilege record (Odoo 19 group privilege system)
  • res.groups: category_idprivilege_id
  • res.groups: usersuser_ids (field renamed in v19)
  • res.users: groups_idgroup_ids (field renamed in v19)
  • Search view inheritance: bare <search> → proper <xpath>
  • ir.actions.act_window: target="inline"target="main"

Testing

Tested on live Odoo 19.0 Docker instance — installs and works correctly.

🤖 Generated with Claude Code

@pedrobaeza
Copy link
Member

Thanks for the contribution.

Please preserve commit history following technical method explained in https://github.com/OCA/maintainer-tools/wiki/Migration-to-version-19.0.

If the jump is between several versions, you have to modify the source branch in the main command to accommodate it to this circumstance.

naousse and others added 27 commits March 16, 2026 11:33
* Copyright and various stuff
* Contributors
* External ID not found in the system: base.group_document_user
Currently translated at 93.9% (31 of 33 strings)

Translation: knowledge-11.0/knowledge-11.0-knowledge
Translate-URL: https://translation.odoo-community.org/projects/knowledge-11-0/knowledge-11-0-knowledge/pt_BR/
Updated by Update PO files to match POT (msgmerge) hook in Weblate.
Otherwise the module grants too wide permissions for all users.
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: knowledge-12.0/knowledge-12.0-knowledge
Translate-URL: https://translation.odoo-community.org/projects/knowledge-12-0/knowledge-12-0-knowledge/
Currently translated at 57.7% (15 of 26 strings)

Translation: knowledge-12.0/knowledge-12.0-knowledge
Translate-URL: https://translation.odoo-community.org/projects/knowledge-12-0/knowledge-12-0-knowledge/fr/
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: knowledge-13.0/knowledge-13.0-knowledge
Translate-URL: https://translation.odoo-community.org/projects/knowledge-13-0/knowledge-13-0-knowledge/
tate11 and others added 21 commits March 16, 2026 11:33
Currently translated at 88.4% (23 of 26 strings)

Translation: knowledge-15.0/knowledge-15.0-knowledge
Translate-URL: https://translation.odoo-community.org/projects/knowledge-15-0/knowledge-15-0-knowledge/tr/
Currently translated at 100.0% (26 of 26 strings)

Translation: knowledge-15.0/knowledge-15.0-knowledge
Translate-URL: https://translation.odoo-community.org/projects/knowledge-15-0/knowledge-15-0-knowledge/fr/
In v16 Odoo has its own knowledge module
Currently translated at 100.0% (21 of 21 strings)

Translation: knowledge-17.0/knowledge-17.0-document_knowledge
Translate-URL: https://translation.odoo-community.org/projects/knowledge-17-0/knowledge-17-0-document_knowledge/sv/
Currently translated at 100.0% (21 of 21 strings)

Translation: knowledge-17.0/knowledge-17.0-document_knowledge
Translate-URL: https://translation.odoo-community.org/projects/knowledge-17-0/knowledge-17-0-document_knowledge/ca/
[IMP] document_knowledge: pre-commit execution
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: knowledge-18.0/knowledge-18.0-document_knowledge
Translate-URL: https://translation.odoo-community.org/projects/knowledge-18-0/knowledge-18-0-document_knowledge/
Currently translated at 100.0% (15 of 15 strings)

Translation: knowledge-18.0/knowledge-18.0-document_knowledge
Translate-URL: https://translation.odoo-community.org/projects/knowledge-18-0/knowledge-18-0-document_knowledge/it/
Currently translated at 100.0% (15 of 15 strings)

Translation: knowledge-18.0/knowledge-18.0-document_knowledge
Translate-URL: https://translation.odoo-community.org/projects/knowledge-18-0/knowledge-18-0-document_knowledge/sv/
@gregory-rutecki gregory-rutecki force-pushed the 19.0-mig-document_knowledge branch from 9fb0535 to 0f712f8 Compare March 16, 2026 10:48
@gregory-rutecki
Copy link
Author

@pedrobaeza Done — I've rebased all PRs to preserve the full commit history from 18.0 as described in the migration guide. All 12 PRs now pass pre-commit. Thanks for the pointer!

@pedrobaeza
Copy link
Member

I'm seeing that #569 already migrated it.

@gregory-rutecki
Copy link
Author

@pedrobaeza I understand there's overlap with #569, but that PR has been open since December 2025 and still hasn't been merged — along with 9 other PRs from the same author that are stuck without review. Meanwhile the entire knowledge module suite remains unavailable for Odoo 19 users.

My set of 12 PRs (#588-#599) provides a complete, tested migration of all knowledge modules to 19.0, including two modules (attachment_preview, document_page_reference) that have no other PRs. All pass pre-commit, all have been tested on a live Odoo 19 instance, and all preserve the commit history from 18.0 as you requested.

I'd rather see the community get a working knowledge suite on 19.0 — whether that's through my PRs or anyone else's. Happy to help however makes sense.

@pedrobaeza
Copy link
Member

If it hasn't been merged because the lack of reviews, but it's correct, what you should do is to review it instead of opening a new one.

@gregory-rutecki
Copy link
Author

Closing in favor of #569 — reviewed and approved the existing PR instead.

@gregory-rutecki
Copy link
Author

@pedrobaeza You're right. I've reviewed and approved all existing PRs from BhaveshHeliconia (#569-#579). Closing my duplicates. Keeping #588 (attachment_preview) and #589 (document_page_reference) as those modules don't have existing PRs.

@pedrobaeza
Copy link
Member

Thanks for your understanding and contribution!

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.