Skip to content

Comments

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

Open
BhaveshHeliconia wants to merge 55 commits intoOCA:19.0from
HeliconiaIO:19.0-mig-document_knowledge
Open

[19.0] [MIG] document_knowledge: Migration to 19.0#569
BhaveshHeliconia wants to merge 55 commits intoOCA:19.0from
HeliconiaIO:19.0-mig-document_knowledge

Conversation

@BhaveshHeliconia
Copy link
Contributor

@BhaveshHeliconia BhaveshHeliconia commented Dec 29, 2025

No description provided.

@BhaveshHeliconia BhaveshHeliconia mentioned this pull request Dec 30, 2025
Open
11 tasks
@stferraro
Copy link

stferraro commented Dec 30, 2025

Hi,

I reviewed the module on 19.0 and all checks are green.
Functionally and technically it looks complete from my side.

Could you please confirm if this PR is ready to be merged, or if any additional changes are required?

Thanks!

@BhaveshHeliconia
Copy link
Contributor Author

BhaveshHeliconia commented Dec 31, 2025

@stferraro Thanks for reviewing the PR and leaving comments. However, comments alone don’t mark the PR as approved. To approve it, please go to the Files changed tab, click the green Review changes button, select Approve if everything looks fine, and then submit. That’s the process to approve a PR.

  • From my side, no further changes are required. Once we receive approval from the maintainer, the PR can be merged.

@OCA-git-bot
Copy link
Contributor

OCA-git-bot commented Jan 7, 2026

This PR has the approved label and has been created more than 5 days ago. It should therefore be ready to merge by a maintainer (or a PSC member if the concerned addon has no declared maintainer). 🤖

@BhaveshHeliconia BhaveshHeliconia mentioned this pull request Jan 12, 2026
Open
farhatzafar
farhatzafar suggested changes Feb 20, 2026
View reviewed changes
Copy link

@farhatzafar farhatzafar left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

In document_knowledge/demo/document_knowledge.xml: groups_id should be changed to group_ids. Other than this, I have tested it on Odoo 19 and the migration looks complete.

naousse and others added 18 commits February 23, 2026 11:39
@naousse @BhaveshHeliconia
knowledge module added from Odoo 8.0 and ported to 9.0
1844472
@BhaveshHeliconia
Add CMIS modules to Settings
608bda8
@LeartS @BhaveshHeliconia
Port to v10
893e867
@BhaveshHeliconia
[FIX] knowledge: Several things
6b6cbff 
* Copyright and various stuff
* Contributors
* External ID not found in the system: base.group_document_user
@MiquelRForgeFlow @BhaveshHeliconia
knowledge: migrations scripts
4380806
@fayezqandeel @BhaveshHeliconia
knowledge: Migration to 11.0
d73cae4
@cubells @BhaveshHeliconia
[MIG] knowledge: Complete migration
35d68e3
@cubells @BhaveshHeliconia
[IMP] knowledge: add missing menus
c08149c
@BhaveshHeliconia
Translated using Weblate (Portuguese (Brazil))
2f8da3f 
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/
@nicomacr @BhaveshHeliconia
[FIX] remove the set and get values in res.config
c34d563
@SalahAdDin @BhaveshHeliconia
Fixing items' link error.
9d1b561 
It solves this: OCA#179
@ivantodorovich @BhaveshHeliconia
Enterprise-like icon
e77e6f5
@ivantodorovich @BhaveshHeliconia
[knowledge] Consider the module as a fully-fledge application
9840d57
@ivantodorovich @BhaveshHeliconia
Add icon svg master file
983cb6b
@oca-transbot @BhaveshHeliconia
Update translation files
2b80131 
Updated by Update PO files to match POT (msgmerge) hook in Weblate.
@JordiBForgeFlow @BhaveshHeliconia
[knowledge][11.0.3.0.0] - add security for access to ir.attachment.
ae1287f 
Otherwise the module grants too wide permissions for all users.
@MiquelRForgeFlow @BhaveshHeliconia
[FIX] pylint/javascript-lint
0192672
@MiquelRForgeFlow @BhaveshHeliconia
[MIG] knowledge: Migration to 12.0
90912bc
victoralmau and others added 22 commits February 23, 2026 11:39
@victoralmau @BhaveshHeliconia
[FIX] knowledge: The internal user group is added as an inheritance t…
501696c 
…o the new group.
@OCA-git-bot @BhaveshHeliconia
knowledge 15.0.1.0.1
f8df97c
@tate11 @BhaveshHeliconia
Translated using Weblate (Turkish)
89699d8 
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/
@vincent-hatakeyama @BhaveshHeliconia
Translated using Weblate (French)
0b4554d 
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/
@marielejeune @BhaveshHeliconia
[MIG] knowledge: Migration to 16.0
53805dd
@marielejeune @BhaveshHeliconia
[CHG] module name: knowledge -> document_knowledge
ca21b11 
In v16 Odoo has its own knowledge module
@weblate @BhaveshHeliconia
[16.0][FIX] document_knowledge: Error in data-key res_config
f4e7741
@m1k3lm @BhaveshHeliconia
[IMP] document_knowledge: pre-commit auto fixes
48f6c05
@m1k3lm @BhaveshHeliconia
[MIG] document_knowledge: Migration to 17.0
b17227f
@jakobkrabbe @BhaveshHeliconia
Translated using Weblate (Swedish)
9d7f3c1 
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/
@esteveblanch @BhaveshHeliconia
Translated using Weblate (Catalan)
2b32b55 
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/
@Kimkhoi3010 @BhaveshHeliconia
[IMP] document_knowleadge: pre-commit auto fixes
8b29181 
[IMP] document_knowledge: pre-commit execution
@Kimkhoi3010 @BhaveshHeliconia
[MIG] document_knowledge: Migration to 18.0
9efe03f
@BhaveshHeliconia
[UPD] Update document_knowledge.pot
235ae69
@OCA-git-bot @BhaveshHeliconia
[BOT] post-merge updates
27bc68f
@weblate @BhaveshHeliconia
Update translation files
cf84629 
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/
@mymage @BhaveshHeliconia
Translated using Weblate (Italian)
2b8a9ce 
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/
@nicomacr @BhaveshHeliconia
[FIX] document_knowledge: Use different path to avoid collision with …
fcc82a0 
…odoo documents
@OCA-git-bot @BhaveshHeliconia
[BOT] post-merge updates
700e67e
@jakobkrabbe @BhaveshHeliconia
Translated using Weblate (Swedish)
e9be2bb 
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/
@etobella @BhaveshHeliconia
[IMP] document_knowledge: Improve logo
8d7bcd8
@OCA-git-bot @BhaveshHeliconia
[BOT] post-merge updates
46aa8dd
@BhaveshHeliconia BhaveshHeliconia force-pushed the 19.0-mig-document_knowledge branch from 666a01d to 3457a90 Compare February 23, 2026 06:09
@BhaveshHeliconia BhaveshHeliconia force-pushed the 19.0-mig-document_knowledge branch from 3457a90 to 14e16ed Compare February 23, 2026 06:11
@BhaveshHeliconia
Copy link
Contributor Author

BhaveshHeliconia commented Feb 23, 2026

@farhatzafar Fixed groups_id → group_ids

@OCA-git-bot
Copy link
Contributor

OCA-git-bot commented Feb 23, 2026

This PR has the approved label and has been created more than 5 days ago. It should therefore be ready to merge by a maintainer (or a PSC member if the concerned addon has no declared maintainer). 🤖

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

Reviewers

3 more reviewers

@stferraro stferraro stferraro approved these changes

@CristianoMafraJunior CristianoMafraJunior CristianoMafraJunior approved these changes

@farhatzafar farhatzafar farhatzafar approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

approved ready to merge

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.