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] account_analytic_required: Migration to 18.0 #708

Open
wants to merge 69 commits into
base: 18.0
Choose a base branch
from

Conversation

peluko00
Copy link
Contributor

@peluko00 peluko00 commented Nov 7, 2024

Alexis de Lattre and others added 8 commits November 7, 2024 14:46
…bit=0.0

This is useful in some situations where the tax generates move lines with credit=debit=0.0 with a non-zero tax amount (for instance with tax code VAT-IN-V82-21-EU-S in l10n_be). Without this patch the transaction would be rejected when analytic policy is 'required', since analytic accounts are not propagated through taxes in 6.1.
[RFR] Line length of affected code
[UPD] Update pot file with newly generated code strings
[ADD] Dutch translations
@peluko00 peluko00 mentioned this pull request Nov 7, 2024
9 tasks
@BernatObrador
Copy link
Contributor

LGTM, tested in runboat and code review, but please squash administrative commits.

@peluko00 peluko00 force-pushed the 18.0-mig-account_analytic_required branch from ae55b3e to b2c69d6 Compare November 8, 2024 14:44
OCA-git-bot and others added 27 commits November 8, 2024 15:47
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: account-analytic-13.0/account-analytic-13.0-account_analytic_required
Translate-URL: https://translation.odoo-community.org/projects/account-analytic-13-0/account-analytic-13-0-account_analytic_required/
Currently translated at 100.0% (12 of 12 strings)

Translation: account-analytic-14.0/account-analytic-14.0-account_analytic_required
Translate-URL: https://translation.odoo-community.org/projects/account-analytic-14-0/account-analytic-14-0-account_analytic_required/ca/
Currently translated at 100.0% (12 of 12 strings)

Translation: account-analytic-14.0/account-analytic-14.0-account_analytic_required
Translate-URL: https://translation.odoo-community.org/projects/account-analytic-14-0/account-analytic-14-0-account_analytic_required/es/
Currently translated at 100.0% (12 of 12 strings)

Translation: account-analytic-16.0/account-analytic-16.0-account_analytic_required
Translate-URL: https://translation.odoo-community.org/projects/account-analytic-16-0/account-analytic-16-0-account_analytic_required/pt_BR/
Currently translated at 100.0% (12 of 12 strings)

Translation: account-analytic-16.0/account-analytic-16.0-account_analytic_required
Translate-URL: https://translation.odoo-community.org/projects/account-analytic-16-0/account-analytic-16-0-account_analytic_required/es/
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: account-analytic-16.0/account-analytic-16.0-account_analytic_required
Translate-URL: https://translation.odoo-community.org/projects/account-analytic-16-0/account-analytic-16-0-account_analytic_required/
Currently translated at 100.0% (12 of 12 strings)

Translation: account-analytic-16.0/account-analytic-16.0-account_analytic_required
Translate-URL: https://translation.odoo-community.org/projects/account-analytic-16-0/account-analytic-16-0-account_analytic_required/sv/
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: account-analytic-17.0/account-analytic-17.0-account_analytic_required
Translate-URL: https://translation.odoo-community.org/projects/account-analytic-17-0/account-analytic-17-0-account_analytic_required/
@peluko00 peluko00 force-pushed the 18.0-mig-account_analytic_required branch from b2c69d6 to aff1e62 Compare November 8, 2024 14:47
@peluko00
Copy link
Contributor Author

peluko00 commented Nov 8, 2024

LGTM, tested in runboat and code review, but please squash administrative commits.

Done. thanks!

Copy link

@ppyczko ppyczko left a comment

Choose a reason for hiding this comment

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

LGTM, tested in runboat

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.