Skip to content

[18.0] [MIG] l10n_it_account_vat_period_end_settlement#4675

Merged
OCA-git-bot merged 89 commits into
OCA:18.0from
odooNextev:18.0-mig-account_vat_period_end_statement
May 23, 2025
Merged

[18.0] [MIG] l10n_it_account_vat_period_end_settlement#4675
OCA-git-bot merged 89 commits into
OCA:18.0from
odooNextev:18.0-mig-account_vat_period_end_statement

Conversation

@odooNextev

Copy link
Copy Markdown
Contributor

No description provided.

@primes2h

Copy link
Copy Markdown
Contributor

/ocabot migration account_vat_period_end_statement

@OCA-git-bot OCA-git-bot added this to the 18.0 milestone Mar 26, 2025
@stenext stenext force-pushed the 18.0-mig-account_vat_period_end_statement branch 3 times, most recently from 5c9701f to 93d2e98 Compare March 26, 2025 14:52

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

jado95 added a commit to DinamicheAziendali/l10n-italy that referenced this pull request Mar 28, 2025
l10n_it_appointment_code: OCA#4665
l10n_it_vat_registries: OCA#4664
l10n_it_vat_statement_communication: OCA#4676
@odooNextev

Copy link
Copy Markdown
Contributor Author

#4510

Rinomino il modulo in account_vat_period_end_settlement oltre ai riferimenti nel codice?

@primes2h

primes2h commented Apr 1, 2025

Copy link
Copy Markdown
Contributor

#4510

Rinomino il modulo in account_vat_period_end_settlement oltre ai riferimenti nel codice?

In questo caso potremmo anche approfittare dell'occasione per uniformare e semplificare il nome.

l10n_it_vat_settlement

@OCA/local-italy-developers che ne pensate?

@eLBati

eLBati commented Apr 3, 2025

Copy link
Copy Markdown
Member

@odooNextev questa PR sostituisce questa #4565 ?

@odooNextev

Copy link
Copy Markdown
Contributor Author

@odooNextev questa PR sostituisce questa #4565 ?

@eLBati

eLBati commented Apr 4, 2025

Copy link
Copy Markdown
Member

@odooNextev questa PR sostituisce questa #4565 ?

Nel senso che la #4565 era incompleta?

@odooNextev

Copy link
Copy Markdown
Contributor Author

@odooNextev questa PR sostituisce questa #4565 ?

Nel senso che la #4565 era incompleta?

A dire il vero la #4565 non era segnalata nella issue base di migrazione e non l'avevo trovata, così ne ho cominciata una nuova

@francesco-ooops

Copy link
Copy Markdown
Contributor

includere #4204

@stenext stenext force-pushed the 18.0-mig-account_vat_period_end_statement branch from 93d2e98 to 1f9ae5d Compare April 11, 2025 08:21
Borruso added a commit to DinamicheAziendali/l10n-italy that referenced this pull request Apr 18, 2025


class VatPeriodEndStatementReport(models.AbstractModel):
_name = "report.account_vat_period_end_statement.vat_statement"

@micheledic micheledic Apr 23, 2025

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.

Rinominare in
_name = "report.l10n_it_account_vat_period_end_settlement.vat_statement"
altrimenti non funziona

"assets": {
"web.assets_backend": [
"l10n_it_account_vat_period_end_settlement/static/src/css/"
"l10n_it_account_vat_period_end_settlement.css",

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.

Cambiare in static/css il css in
l10n_it_account_vat_period_end_settlement.css

@MaurizioConte

Copy link
Copy Markdown

Ciao,
ho provato a fare i test su "runboat" ma su entrambi i database ci sono problemi, sul primo DB in elenco mi viene presentato una videata che tutto è fuorché Odoo mentre sul secondo DB accedo ma quando calcolo la liquidazione esce vuota.

Ho provato a testarlo sul mio DB che è già alla 18.0, migrato dalla 16, e c'è un errore in quanto viene esposta solo l'imposta di acquisto con l'imponibile e senza valore dell'iva mentre per la parte di vendita l'imposta non figura proprio.
Essendo l'imposta unica mentre i conti contabili iva utilizzati sono 2 non Riesco a capire come faremo a considerare in liquidazione iva la metodologia di integrazione iva.
Anche perché i dati delle operazioni intra ed extra vanno comunicati all'ADE in LIPE - Comunicazione Dati IVA.

Attendo un vostro riscontro magari anche per confrontarci.
Saluti

@francesco-ooops

Copy link
Copy Markdown
Contributor

In effetti il runboat in questa PR è spaccato

The style compilation failed. This is an administrator or developer error that must be fixed for the entire database before continuing working. See browser console or server logs for details.

@eLBati eLBati force-pushed the 18.0-mig-account_vat_period_end_statement branch 2 times, most recently from 61bd0be to 0bad50d Compare May 23, 2025 10:02
eLBati added 5 commits May 23, 2025 12:12
- Adding _get_credit_accounts and _get_debit_accounts
- Compute deductible_balance in case of split payment
and comments
- Add customer_balance and supplier_balance for VAT documents
- Remove vat_statement_account_id
- Adapt to reverse charge cases
- Adapt to split payment cases
- Fix reconciliation process
- REF tests
@eLBati eLBati force-pushed the 18.0-mig-account_vat_period_end_statement branch from 0bad50d to 995ce8f Compare May 23, 2025 10:12

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

Funzionale OK!!!!

@stenext stenext 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

@eLBati

eLBati commented May 23, 2025

Copy link
Copy Markdown
Member

Metto patch visto che ci sono anche correzioni ad altri moduli

/ocabot merge patch

@OCA-git-bot

Copy link
Copy Markdown
Contributor

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

@OCA-git-bot OCA-git-bot merged commit cdfc8ff into OCA:18.0 May 23, 2025
7 checks passed
@OCA-git-bot

Copy link
Copy Markdown
Contributor

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