Skip to content

[19.0][MIG] l10n_it_edi_doi_extension: Migration to 19.0#5082

Open
Borruso wants to merge 7 commits into
OCA:19.0from
Borruso:19.0-mig-l10n_it_edi_doi_extension
Open

[19.0][MIG] l10n_it_edi_doi_extension: Migration to 19.0#5082
Borruso wants to merge 7 commits into
OCA:19.0from
Borruso:19.0-mig-l10n_it_edi_doi_extension

Conversation

@Borruso

@Borruso Borruso commented Feb 6, 2026

Copy link
Copy Markdown
Contributor

@tafaRU

tafaRU commented Feb 6, 2026

Copy link
Copy Markdown
Member

/ocabot migration l10n_it_edi_doi_extension

@OCA-git-bot OCA-git-bot added this to the 19.0 milestone Feb 6, 2026
@OCA-git-bot OCA-git-bot mentioned this pull request Feb 6, 2026
30 tasks
@Borruso Borruso force-pushed the 19.0-mig-l10n_it_edi_doi_extension branch 2 times, most recently from d8acec2 to 3368826 Compare February 7, 2026 12:18
@Borruso Borruso force-pushed the 19.0-mig-l10n_it_edi_doi_extension branch 2 times, most recently from 4801b22 to 8dab4bf Compare March 15, 2026 10:43
if not doi_tax:
continue
declaration_tax_lines = order.order_line.filtered(
lambda line, doi_tax=doi_tax: doi_tax in line.taxes_id

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

il campo si chiama tax_ids non esiste più taxes_id

doi_tax.name,
),
)
if any(line.taxes_id != doi_tax for line in declaration_tax_lines):

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

il campo su 19.0 è tax_ids non più taxes_id

continue

order_lines = order.order_line.filtered(
lambda line: line.taxes_id.ids == tax.ids

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

come sopra, il campo è tax_ids

order.l10n_it_edi_doi_warning = ""
declaration = order.l10n_it_edi_doi_id

show_warning = declaration and order.state != "cancelled"

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

lo stato sul po è cancel non più cancelled

@micheledic

Copy link
Copy Markdown
Contributor

io proporrei di aggiungere a questa PR anche delle migration per portare le vecchie doi al nativo

Borruso and others added 3 commits June 12, 2026 12:25
…her taxes

This PR is created to accept other taxes than DOI(for example inarcassa)
removed also on odoo core odoo/odoo@b88caf8
@Borruso Borruso force-pushed the 19.0-mig-l10n_it_edi_doi_extension branch from 8dab4bf to 810aea6 Compare June 12, 2026 10:26
@OCA-git-bot OCA-git-bot added mod:l10n_it_edi_doi_extension Module l10n_it_edi_doi_extension series:19.0 labels Jun 12, 2026
@Borruso Borruso requested a review from micheledic June 12, 2026 10:26
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.

7 participants