Skip to content

[MIG] l10n_it_accompanying_invoice: Migration to 18.0#4818

Merged
OCA-git-bot merged 7 commits into
OCA:18.0from
monen17:18.0-mig-l10n_it_accompanying_invoice
Sep 12, 2025
Merged

[MIG] l10n_it_accompanying_invoice: Migration to 18.0#4818
OCA-git-bot merged 7 commits into
OCA:18.0from
monen17:18.0-mig-l10n_it_accompanying_invoice

Conversation

@monen17

@monen17 monen17 commented Jun 27, 2025

Copy link
Copy Markdown
Contributor

Migrazione da 16.0 (https://github.com/OCA/l10n-italy/tree/9e15883c29cbcd594d7c546654daf55a1a7d1575/l10n_it_accompanying_invoice).

Il primo commit modifica un report di l10n_it_delivery_note perché nella migrazione a 18.0 (#4413) è stato modificato.
La differenza si vede confrontando ad esempio questa riga del file di 16.0:

l10n-italy/l10n_it_delivery_note/report/delivery_data.xml

Line 9 in 9e15883

<div t-if="doc.delivery_transport_reason_id" class="col-3 bm-2">

con la stessa riga di 18.0:

l10n-italy/l10n_it_delivery_note/report/delivery_data.xml

Line 9 in 2de18b1

<div class="col" t-if="doc.transport_reason_id" name="transport_reason">

@OCA-git-bot

OCA-git-bot commented Jun 27, 2025

Copy link
Copy Markdown
Contributor

Hi @aleuffre, @MarcoCalcagni, @renda-dev,
some modules you are maintaining are being modified, check this out!

@monen17 monen17 force-pushed the 18.0-mig-l10n_it_accompanying_invoice branch 2 times, most recently from 798a9fe to 8efe26b Compare June 27, 2025 13:25
@monen17

monen17 commented Jun 27, 2025

Copy link
Copy Markdown
Contributor Author

/ocabot migration l10n_it_accompanying_invoice

@OCA-git-bot OCA-git-bot added this to the 18.0 milestone Jun 27, 2025
@OCA-git-bot OCA-git-bot mentioned this pull request Jun 27, 2025
Open
46 tasks
@monen17 monen17 force-pushed the 18.0-mig-l10n_it_accompanying_invoice branch 2 times, most recently from b2887f9 to 02f9c7a Compare June 27, 2025 13:37
@monen17 monen17 marked this pull request as ready for review June 27, 2025 13:37
LorenzoC0
LorenzoC0 approved these changes Jul 3, 2025
View reviewed changes

@LorenzoC0 LorenzoC0 left a comment

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.

LGTM 👍

@Borruso Borruso mentioned this pull request Jul 14, 2025
Open
10 tasks
Borruso added a commit to DinamicheAziendali/l10n-italy that referenced this pull request Jul 14, 2025
@Borruso
[ADD] l10n_it_accompanying_invoice: aggiunta PR OCA#4818
9bfe3ef
@monen17

monen17 commented Aug 2, 2025

Copy link
Copy Markdown
Contributor Author

@Borruso @MarcoCalcagni ho visto che l'avete aggiunta al branch 18.0 del vostro fork in #4818 (reference), l'avete provata?
Lasciate una revisione per aiutare a capire se va bene o no?

@eLBati

eLBati commented Sep 5, 2025

Copy link
Copy Markdown
Member

/ocabot rebase

monen17 and others added 7 commits September 5, 2025 12:10
@monen17 @OCA-git-bot
[FIX] l10n_it_delivery_note: Compatibility with mixin
cb1cc72
@fredzamoabg @OCA-git-bot
ADD l10n_it_fattura_accompagnatoria from openerp-italia (renamed to l…
657497a 
…10n_it_accompanying_invoice)
@SirTakobi @OCA-git-bot
Manually fix manifest website key
33039dc
@SirAionTech @OCA-git-bot
[IMP] l10n_it_accompanying_invoice: pre-commit execution
81bac3d
@SirAionTech @odooNextev @OCA-git-bot
[MIG] l10n_it_accompanying_invoice: Migration to 16.0
dcdd61a 
Co-authored-by: Simone Rubino <simone.rubino@aion-tech.it>
Co-authored-by: OdooNextev <odoo@nextev.it>
@mymage @OCA-git-bot
Translated using Weblate (Italian)
2860a1f 
Currently translated at 100.0% (33 of 33 strings)

Translation: l10n-italy-16.0/l10n-italy-16.0-l10n_it_accompanying_invoice
Translate-URL: https://translation.odoo-community.org/projects/l10n-italy-16-0/l10n-italy-16-0-l10n_it_accompanying_invoice/it/
@monen17 @OCA-git-bot
[MIG] l10n_it_accompanying_invoice: Migration to 18.0
f19eaf1
@OCA-git-bot

OCA-git-bot commented Sep 5, 2025

Copy link
Copy Markdown
Contributor

Congratulations, PR rebased to 18.0.

@OCA-git-bot OCA-git-bot force-pushed the 18.0-mig-l10n_it_accompanying_invoice branch from 02f9c7a to f19eaf1 Compare September 5, 2025 12:11
mrcast
mrcast approved these changes Sep 5, 2025
View reviewed changes

@mrcast mrcast left a comment

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.

Test funzionale: OK!

@OCA-git-bot

OCA-git-bot commented Sep 5, 2025

Copy link
Copy Markdown
Contributor

This PR has the approved label and has been created more than 5 days ago. It should therefore be ready to merge by a maintainer (or a PSC member if the concerned addon has no declared maintainer). 🤖

Borruso added a commit to DinamicheAziendali/l10n-italy that referenced this pull request Sep 9, 2025
@Borruso
[UPD] l10n_it_accompanying_invoice: aggiornato PR OCA#4818
09322e9
@eLBati

eLBati commented Sep 12, 2025

Copy link
Copy Markdown
Member

/ocabot merge nobump

@OCA-git-bot

OCA-git-bot commented Sep 12, 2025

Copy link
Copy Markdown
Contributor

This PR looks fantastic, let's merge it!
Prepared branch 18.0-ocabot-merge-pr-4818-by-eLBati-bump-nobump, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit 6302f72 into OCA:18.0 Sep 12, 2025
7 checks passed
@OCA-git-bot

OCA-git-bot commented Sep 12, 2025

Copy link
Copy Markdown
Contributor

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

Reviewers

2 more reviewers

@mrcast mrcast mrcast approved these changes

@LorenzoC0 LorenzoC0 LorenzoC0 approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

approved merged 🎉 migration ready to merge

Projects

None yet

Milestone

18.0

Development

Successfully merging this pull request may close these issues.

10 participants

@monen17 @OCA-git-bot @eLBati @mrcast @LorenzoC0 @francesco-ooops @fredzamoabg @SirTakobi @SirAionTech @mymage
⚡