Repository navigation
[20.0][MIG] document_knowledge: Migration to 20.0 - #646
Conversation
* 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/
It solves this: OCA#179
Otherwise the module grants too wide permissions for all users.
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/
Currently translated at 57.7% (15 of 26 strings) Translation: knowledge-13.0/knowledge-13.0-knowledge Translate-URL: https://translation.odoo-community.org/projects/knowledge-13-0/knowledge-13-0-knowledge/he_IL/
Currently translated at 100.0% (26 of 26 strings) Translation: knowledge-13.0/knowledge-13.0-knowledge Translate-URL: https://translation.odoo-community.org/projects/knowledge-13-0/knowledge-13-0-knowledge/ru/
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/
|
We're running |
|
This PR has the |
Head 8533215 of ledoent:20.0-mig-document_knowledge, open, approved, CI green.
yann-logisteme
left a comment
yann-logisteme
left a comment
There was a problem hiding this comment.
Tested on Odoo 20.0 Community (image of 2026-09-26) together with #647: clean install, no warning, document_page tests pass (18), and we now use it on our staging instance.
@etobella about your remark: this PR adds no model (only ir.attachment and res.config.settings inherits), so there is no ir.access to declare here. The security file you were looking for is in #647. Could you merge this one? #647 and #648 are only waiting for it.
Assisted-by: Claude Opus 5.5
|
Can you edit last comment to review CLA @dnplkndll ? |
@etobella you are just asking to retrigger for the CLA message right? |
Signed-off-by: Don Kendall <[email protected]> Assisted-by: Claude Opus 5
8533215 to
bfc5f48
Compare
|
The following commit authors need to sign the Contributor License Agreement: |
bm1-phillip
left a comment
bm1-phillip
left a comment
There was a problem hiding this comment.
Functional review on Odoo 20.0 Community (nightly 20261110), l10n_de (SKR03), together with #647. We use this module in production on 19.0 as the base of our internal handbook.
It installs without warnings in the log (the module has no tests of its own), and document_page on top of it works as described in #647. LGTM.
|
@etobella OK to merge, as CLA Signing is blocking a lot of PRs right now and is not actually mandatory? |
|
Yes, we can proceed. |
|
/ocabot merge nobump |
|
@StefanRijnhart The merge process could not start, because command |
|
@etobella Got this error before on another PR today. Do you know where it originates? |
|
Bot failed today. I think it is fixed now /ocabot merge nobump |
|
Hey, thanks for contributing! Proceeding to merge this for you. |
|
Congratulations, your PR was merged at af6c71e. Thanks a lot for contributing to OCA. ❤️ |
|
@etobella Thanks! |
|
/ocabot migration document_knowledge |
Port of
document_knowledgefrom19.0to20.0. Follows the OCA migration guide.First addon on the
20.0branch; it is the base of thedocument_pagefamily, so it lands first.AI-assisted (Claude Code); every change reviewed, tested, and owned by the author.