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

[MIG] report_qweb_operating_unit: Migration to 17.0 #677

Open
wants to merge 27 commits into
base: 17.0
Choose a base branch
from

Conversation

jdidderen-nsi
Copy link
Contributor

@jdidderen-nsi jdidderen-nsi commented May 30, 2024

In the case of this migration, I extended the module to have the same layouts as the standard ones and almost the same capabilities as a company.

IMO a potential improvement would be to have a layout by operating unit and not just by company.

Depends on:

@AaronHForgeFlow
Copy link
Contributor

Thank you for the work! LG!

serpentcs-dev1 and others added 25 commits July 8, 2024 21:42
Currently translated at 100.0% (10 of 10 strings)

Translation: operating-unit-13.0/operating-unit-13.0-report_qweb_operating_unit
Translate-URL: https://translation.odoo-community.org/projects/operating-unit-13-0/operating-unit-13-0-report_qweb_operating_unit/es_MX/
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: operating-unit-14.0/operating-unit-14.0-report_qweb_operating_unit
Translate-URL: https://translation.odoo-community.org/projects/operating-unit-14-0/operating-unit-14-0-report_qweb_operating_unit/
@jdidderen-nsi jdidderen-nsi force-pushed the 17.0-mig-report_qweb_operating_unit branch from e455e91 to 5708d4f Compare July 8, 2024 19:42
Copy link
Member

@hbrunn hbrunn left a comment

Choose a reason for hiding this comment

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

great initiative with the report fields on OUs, I base the v16 migration on your work.

Please cherry pick a82200d for tests

@jdidderen-nsi
Copy link
Contributor Author

great initiative with the report fields on OUs, I base the v16 migration on your work.

Please cherry pick a82200d for tests

@hbrunn And it's cherry-picked. I only adapted a bit the unit test to use the Operating Unit common class.

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.