Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[18.0][MIG] document_page: Migration to 18.0 #504

Merged
merged 140 commits into from
Nov 30, 2024

Conversation

jdoutreloux
Copy link

@jdoutreloux jdoutreloux commented Nov 7, 2024

Depends on #503

Maxime Chambreuil and others added 30 commits November 7, 2024 16:57
* Add dependency to knowledge
* Adding hack in document_page_history to allow calling method that don't exists
* document_page_history don't have _sql attribute
* Filter more fields
* Changed history widget to html.
* Improved views and added followers to pages.
* Updated document_page pot and es translations.
* document_page:^Cissing dependency.
* [MIG] Migration to version 10.0
* [FIX] Comments and CI errors
* [FIX] ValueError: External ID not found in the system: base.menu_base_partner
* [FIX] ValueError: Wrong value for ir.actions.act_window.target: 'inlineview'
* [FIX] Based on @lasley comments
* [FIX] External ID not found in the system: base.group_document_user
Currently translated at 51.9% (28 of 54 strings)

Translation: knowledge-11.0/knowledge-11.0-document_page
Translate-URL: https://translation.odoo-community.org/projects/knowledge-11-0/knowledge-11-0-document_page/fa/
Currently translated at 77.8% (42 of 54 strings)

Translation: knowledge-11.0/knowledge-11.0-document_page
Translate-URL: https://translation.odoo-community.org/projects/knowledge-11-0/knowledge-11-0-document_page/de/
Fixing `Page` parents.
…Add history name field, to name revisions. Modified views to allow setting this field.

[document_page] FIX BUG: UI hanging when editing content. This was due to the api.depends on _compute_diff. Removing it because it's not really necessary since the field is not stored.

[document_page] page_id should be readonly.

Improve active field

[document_page] Update version number

[UPD] Update document_page.pot

[MIG] document_page_multi_company (OCA#188)

* [MIG] document_page_multi_company

Added this feature from the old module directly in document_page
- fix references: report module is now web module

- display_* fields are removed in this version
 * move menus to a higher level.
 * fix categories search view.
 * fix create menu action not being accessible in any way.
 * reorder form view, focusing in the content, not in secondary fields.
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: knowledge-12.0/knowledge-12.0-document_page
Translate-URL: https://translation.odoo-community.org/projects/knowledge-12-0/knowledge-12-0-document_page/
[UPD] README.rst

document_page 12.0.1.1.0

Update translation files

Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: knowledge-12.0/knowledge-12.0-document_page
Translate-URL: https://translation.odoo-community.org/projects/knowledge-12-0/knowledge-12-0-document_page/

[UPD] README.rst
@TDu
Copy link
Member

TDu commented Nov 12, 2024

@jdoutreloux Thanks for the migration it looks good.

To fix the tests and runboat, you need to mention the dependency because it is not merged yet, following this https://github.com/OCA/maintainer-tools/wiki/Use-temporary-reference(s)-to-another-pull-request(s)

@jdoutreloux jdoutreloux force-pushed the 18.0-mig-document-page branch 2 times, most recently from 4891a19 to 86803a3 Compare November 12, 2024 12:46
@TDu
Copy link
Member

TDu commented Nov 14, 2024

@jdoutreloux #503 has been merged could you drop the commit [DON'T MERGE] test-requirements.txt
Thanks

@natuan9
Copy link

natuan9 commented Nov 14, 2024

Hi @jdoutreloux ,
There are several PRs in 14.0 that are not included in 17.0. I created PR to port them to 17.0

Now, I create this PR to your branch for migration to 18.0

@jdoutreloux
Copy link
Author

@jdoutreloux #503 has been merged could you drop the commit [DON'T MERGE] test-requirements.txt Thanks

I've removed the commit

@max3903
Copy link
Member

max3903 commented Nov 30, 2024

/ocabot merge nobump

@OCA-git-bot
Copy link
Contributor

This PR looks fantastic, let's merge it!
Prepared branch 18.0-ocabot-merge-pr-504-by-max3903-bump-nobump, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit e2cee29 into OCA:18.0 Nov 30, 2024
7 checks passed
@OCA-git-bot
Copy link
Contributor

Congratulations, your PR was merged at 1b8a99c. Thanks a lot for contributing to OCA. ❤️

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

Successfully merging this pull request may close these issues.