Skip to content

[18.0] [MIG] l10n_it_bill_of_entry#4660

Merged
OCA-git-bot merged 28 commits into
OCA:18.0from
odooNextev:18.0-mig-l10n_it_bill_of_entry
Apr 18, 2025
Merged

[18.0] [MIG] l10n_it_bill_of_entry#4660
OCA-git-bot merged 28 commits into
OCA:18.0from
odooNextev:18.0-mig-l10n_it_bill_of_entry

Conversation

@odooNextev

@odooNextev odooNextev commented Mar 19, 2025

Copy link
Copy Markdown
Contributor

Odoo SA non supporta la gestione delle bolle doganali e non è neanche riportata nella roadmap perciò ho migrato il modulo OCA.

@odooNextev odooNextev mentioned this pull request Mar 19, 2025
46 tasks
@stenext stenext force-pushed the 18.0-mig-l10n_it_bill_of_entry branch 2 times, most recently from 61bd3bb to 8398781 Compare March 19, 2025 14:51
@eLBati

eLBati commented Mar 21, 2025

Copy link
Copy Markdown
Member

/ocabot migration l10n_it_bill_of_entry

@OCA-git-bot OCA-git-bot added this to the 18.0 milestone Mar 21, 2025
@stenext stenext force-pushed the 18.0-mig-l10n_it_bill_of_entry branch from 8398781 to 9687acd Compare March 26, 2025 13:17
@stenext stenext force-pushed the 18.0-mig-l10n_it_bill_of_entry branch from bae6cbd to efaef48 Compare April 10, 2025 15:30
tafaRU and others added 23 commits April 11, 2025 11:52
[FIX] mail author

[ADD] advance_customs_vat and wizard/select_template

[FIX] Add __init__.py

[ADD] button 'Make Bill of Entry'

[FIX] Remove pdb

[REF] override load_template

[ADD] check if invoice.customs_doc_type == 'forwarder_invoice'
[fix] osv -> orm

[add] action_move_create

[add] company

[add] storno move

[add] reconciliation

[add] action_cancel

[imp] descr

[imp] company view

[fix] view

[fix] view

[fix] view

[fix] errore msg

[fix] m2m

[fix] missing partner

[imp] tax code

[imp] description

[add] allow to link BOE to forwarder inv

[imp] interface

[imp] interface

[add] trans

[imp] interface

[REF] Set all the modules of the master branches to "installable": False
moves all modules into a __unported__ directory
 * essential features migration from rel. 7
 * adding demo data and unit tests
Currently translated at 100.0% (46 of 46 strings)

Translation: l10n-italy-14.0/l10n-italy-14.0-l10n_it_bill_of_entry
Translate-URL: https://translation.odoo-community.org/projects/l10n-italy-14-0/l10n-italy-14-0-l10n_it_bill_of_entry/it/
Currently translated at 100.0% (52 of 52 strings)

Translation: l10n-italy-14.0/l10n-italy-14.0-l10n_it_bill_of_entry
Translate-URL: https://translation.odoo-community.org/projects/l10n-italy-14-0/l10n-italy-14-0-l10n_it_bill_of_entry/it/
Currently translated at 100.0% (48 of 48 strings)

Translation: l10n-italy-16.0/l10n-italy-16.0-l10n_it_bill_of_entry
Translate-URL: https://translation.odoo-community.org/projects/l10n-italy-16-0/l10n-italy-16-0-l10n_it_bill_of_entry/it/
SirAionTech and others added 5 commits April 11, 2025 11:52
The user must be logged in only one company for the tests to behave as expected.
For instance, searching for a journal should only find journals in the current company
Currently translated at 100.0% (48 of 48 strings)

Translation: l10n-italy-16.0/l10n-italy-16.0-l10n_it_bill_of_entry
Translate-URL: https://translation.odoo-community.org/projects/l10n-italy-16-0/l10n-italy-16-0-l10n_it_bill_of_entry/it/
@stenext stenext force-pushed the 18.0-mig-l10n_it_bill_of_entry branch from efaef48 to 9677e9a Compare April 11, 2025 09:52

@monen17 monen17 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.

Grazie della PR!
Grande che hai già schiacciato i commit dei bot 👏

Ho fatto revisione del codice e per me è ok.

Il commit 9677e9a è una fix che andrebbe riportata anche in 16.0 o è una cosa solo per 18.0?

"base",
"account",
],
"depends": ["base", "account", "l10n_it"],

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.

La dipendenza da l10n_it in passato andava evitata, ma direi che ormai l'abbiamo sdoganata 😉 (pun intended)
👍

@@ -81,7 +81,9 @@ def setUp(self):
"amount": 22,

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.

Ora che abbiamo l10n_it, non esiste già questa imposta nei dati demo?

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Il 22% utilizzato per le dogane non è lo stesso utilizzato per gli acquisti, quindi è probabile che non ci sia nei dati di demo

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.

Github non me la faceva indicare tutta, ma viene creata così:

self.tax_model.create(
            {
                "name": "22%",
                "amount": 22,
                "amount_type": "percent",
                "type_tax_use": "purchase",
                "tax_group_id": self.env.ref(
                    f"account.{demo_data_company.id}_tax_group_iva_22"
                ).id,
            }
        )

mi sembra sia un'imposta generica per gli acquisti, e potrebbe già esserci.
Se fosse quella per le dogane andrebbe almeno indicato nel nome.

@monen17

monen17 commented Apr 11, 2025

Copy link
Copy Markdown
Contributor

Secondo me ci vuole needs functional review .

@VBNext VBNext left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

LGTM

@odooNextev

Copy link
Copy Markdown
Contributor Author

Grazie della PR! Grande che hai già schiacciato i commit dei bot 👏

Ho fatto revisione del codice e per me è ok.

Il commit 9677e9a è una fix che andrebbe riportata anche in 16.0 o è una cosa solo per 18.0?

Deve essere ancora approvata, c'è il task nelle hotfix

@OCA-git-bot

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). 🤖

@odooNextev

Copy link
Copy Markdown
Contributor Author

Grazie della PR! Grande che hai già schiacciato i commit dei bot 👏
Ho fatto revisione del codice e per me è ok.
Il commit 9677e9a è una fix che andrebbe riportata anche in 16.0 o è una cosa solo per 18.0?

Deve essere ancora approvata, c'è il task nelle hotfix

@monen17 ho rimosso questo commit così si può mergiare la migrazione e poi farò il fw Port della PR

@eLBati

eLBati commented Apr 18, 2025

Copy link
Copy Markdown
Member

/ocabot merge nobump

@OCA-git-bot

Copy link
Copy Markdown
Contributor

What a great day to merge this nice PR. Let's do it!
Prepared branch 18.0-ocabot-merge-pr-4660-by-eLBati-bump-nobump, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit 1bab9d8 into OCA:18.0 Apr 18, 2025
@OCA-git-bot

Copy link
Copy Markdown
Contributor

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