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

[17.0][MIG]account_invoice_fixed_discount: Migrated module to v17.0 #1667

Merged
merged 55 commits into from
Mar 8, 2024

Conversation

fd-oerp
Copy link

@fd-oerp fd-oerp commented Feb 14, 2024

No description provided.

LoisRForgeFlow and others added 30 commits February 14, 2024 11:40
Currently translated at 100.0% (6 of 6 strings)

Translation: account-invoicing-12.0/account-invoicing-12.0-account_invoice_fixed_discount
Translate-URL: https://translation.odoo-community.org/projects/account-invoicing-12-0/account-invoicing-12-0-account_invoice_fixed_discount/de/
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: account-invoicing-13.0/account-invoicing-13.0-account_invoice_fixed_discount
Translate-URL: https://translation.odoo-community.org/projects/account-invoicing-13-0/account-invoicing-13-0-account_invoice_fixed_discount/
Odoo has changed the signature of the method `_get_fields_onchange_balance_model`.
OCA-git-bot and others added 14 commits February 14, 2024 11:40
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: account-invoicing-15.0/account-invoicing-15.0-account_invoice_fixed_discount
Translate-URL: https://translation.odoo-community.org/projects/account-invoicing-15-0/account-invoicing-15-0-account_invoice_fixed_discount/
Currently translated at 100.0% (8 of 8 strings)

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

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

Translation: account-invoicing-16.0/account-invoicing-16.0-account_invoice_fixed_discount
Translate-URL: https://translation.odoo-community.org/projects/account-invoicing-16-0/account-invoicing-16-0-account_invoice_fixed_discount/es/
hr_expense calls _convert_to_tax_base_line_dict with base_line set to None. Comparison of base_line._name in account_invoice_fixed_discount causes an attribute error.

Add extra check to ensure there is an account.move.line to check against in base_line.
@fd-oerp
Copy link
Author

fd-oerp commented Feb 20, 2024

Hi @OCA-git-bot can I have any update regarding this PR ?

@fd-oerp
Copy link
Author

fd-oerp commented Feb 29, 2024

Hi @JordiBForgeFlow , can you let me know if I'm missing anything in this PR?

@fd-oerp
Copy link
Author

fd-oerp commented Mar 4, 2024

Hi @JordiBForgeFlow , can you please review this ?

Copy link

@MohamedOsman7 MohamedOsman7 left a comment

Choose a reason for hiding this comment

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

Code & functionality LGTM

@JordiBForgeFlow
Copy link
Sponsor Member

/ocabot migration account_invoice_fixed_discount

@JordiBForgeFlow
Copy link
Sponsor Member

/ocabot merge nobump

@OCA-git-bot OCA-git-bot added this to the 17.0 milestone Mar 8, 2024
@OCA-git-bot OCA-git-bot mentioned this pull request Mar 8, 2024
40 tasks
@OCA-git-bot
Copy link
Contributor

On my way to merge this fine PR!
Prepared branch 17.0-ocabot-merge-pr-1667-by-JordiBForgeFlow-bump-nobump, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit 250002c into OCA:17.0 Mar 8, 2024
7 checks passed
@OCA-git-bot
Copy link
Contributor

Congratulations, your PR was merged at a54e954. 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.

None yet