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_invoice_fiscal_position_update: Migration to 18.0 #1863

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

Conversation

rov-adhoc
Copy link

No description provided.

alexis-via and others added 30 commits December 9, 2024 16:36
Currently translated at 100,0% (4 of 4 strings)

Translation: account-invoicing-11.0/account-invoicing-11.0-account_invoice_fiscal_position_update
Translate-URL: https://translation.odoo-community.org/projects/account-invoicing-11-0/account-invoicing-11-0-account_invoice_fiscal_position_update/nl_NL/
Currently translated at 100,0% (4 of 4 strings)

Translation: account-invoicing-11.0/account-invoicing-11.0-account_invoice_fiscal_position_update
Translate-URL: https://translation.odoo-community.org/projects/account-invoicing-11-0/account-invoicing-11-0-account_invoice_fiscal_position_update/de/
Currently translated at 100.0% (4 of 4 strings)

Translation: account-invoicing-12.0/account-invoicing-12.0-account_invoice_fiscal_position_update
Translate-URL: https://translation.odoo-community.org/projects/account-invoicing-12-0/account-invoicing-12-0-account_invoice_fiscal_position_update/de/
Currently translated at 100.0% (4 of 4 strings)

Translation: account-invoicing-12.0/account-invoicing-12.0-account_invoice_fiscal_position_update
Translate-URL: https://translation.odoo-community.org/projects/account-invoicing-12-0/account-invoicing-12-0-account_invoice_fiscal_position_update/pt_BR/
Currently translated at 100.0% (4 of 4 strings)

Translation: account-invoicing-13.0/account-invoicing-13.0-account_invoice_fiscal_position_update
Translate-URL: https://translation.odoo-community.org/projects/account-invoicing-13-0/account-invoicing-13-0-account_invoice_fiscal_position_update/nl/
Currently translated at 100.0% (4 of 4 strings)

Translation: account-invoicing-13.0/account-invoicing-13.0-account_invoice_fiscal_position_update
Translate-URL: https://translation.odoo-community.org/projects/account-invoicing-13-0/account-invoicing-13-0-account_invoice_fiscal_position_update/es/
… taxes when this was changed.

Otherwise the taxes are not changed when change the fpos.
Don't create new variant, just use the default one.
Currently translated at 100.0% (6 of 6 strings)

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

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

Translation: account-invoicing-16.0/account-invoicing-16.0-account_invoice_fiscal_position_update
Translate-URL: https://translation.odoo-community.org/projects/account-invoicing-16-0/account-invoicing-16-0-account_invoice_fiscal_position_update/pt_BR/
…fiscal position with a line without product and description
@gonzalolenzi
Copy link

LGTM

@rov-adhoc
Copy link
Author

@pedrobaeza Hi! I would like to ask you if it's something missing in this migration in order to be approve
Thanks in advance!

@pedrobaeza
Copy link
Member

@legalsylvain
Copy link
Contributor

Hi @rov-adhoc thanks for porting this module. could you rebase, to re build runboat. I'd like to know if the bug described here is still present in the 18.0 branch. (#1896)
Thanks !

@rov-adhoc rov-adhoc force-pushed the 18.0-mig-account_invoice_fiscal_position_update branch from 25ebb65 to 0c47ca8 Compare January 30, 2025 10:33
@rov-adhoc
Copy link
Author

@legalsylvain thanks for your message. I was able to reproduce the same error in the 18.0 branch. Do you know if anyone is currently working on a fix for this issue?

@legalsylvain
Copy link
Contributor

Do you know if anyone is currently working on a fix for this issue?

I don't know. I just raised this error, using the V16 module. Let's wait what maintainers and original developpers are saying about that bug.

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.