[14.0][MIG] l10n_it_intrastat #2228

SimoRubi · 2021-04-22T12:36:28Z

Migrazione da https://github.com/OCA/l10n-italy/tree/8d6bb2847a9c3f80e632935cb7d80e69f028876d/l10n_it_intrastat

Dipende da OCA/web#1952

--
Confermo di aver firmato il CLA https://odoo-community.org/page/cla e di aver letto le linee guida su https://odoo-community.org/page/contributing

fredzamoabg · 2021-04-30T06:35:42Z

l10n_it_intrastat/models/account.py

+                if country_id:
+                    country_payment_id = country_id
+        elif self.move_id.is_purchase_document():
+            country_payment_id = self.invoice_id.partner_id.country_id


invoice_id va sostituito con move_id

Grazie, fatto

TheMule71 · 2021-04-30T10:29:30Z

Ciao,
vedo in giro ancora dei size= nei fields.Char()

SimoRubi · 2021-05-06T08:16:46Z

Ciao,
vedo in giro ancora dei size= nei fields.Char()

Sì, a quanto vedo size è sempre stato deprecato fin da odoo/odoo@524bb0a ma funziona e non vedo una valida alternativa: si potrebbero aggiungere dei constraint ma mi pare essere moolto più developer-friendly un semplice size= nella definizione del campo; i constraint li vedo utili per gestire casi più complessi.
Dici che sono da togliere comunque?

Otherwise the following warning is raised: unknown parameter 'size', if this is an actual parameter you may want to override the method _valid_field_parameter on the relevant model in order to allow it

TheMule71

LGTM

OCA-git-bot · 2021-11-15T13:54:03Z

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

tafaRU · 2021-11-15T14:02:02Z

Mergi tu @TheMule71?

TheMule71 · 2021-11-16T13:00:31Z

/ocabot merge nobump

tafaRU · 2021-11-18T11:14:06Z

@TheMule71 il tuo merge non è andato a buon fine: puoi riprovare? Considerato che ci sono altri commit dopo quello di merge, non userei l'opzione nobump ma almeno patch, che ne pensi?

AntonioMariaVigliotti · 2021-11-18T16:15:59Z

Ho creato un cliente tedesco e ho creato una fattura di 5 righe:
2 prodotti intrastat che sono correttamente riportati nella lista intrastat
1 prodotto escluso che non è correttamente riportato nella lista intrastat
1 prodotto di servizio che è correttamente riportato nella lista intrastat
1 prodotto misto ripartito sui prodotti precedenti

Visualizzazione dati Intrastat in fattura molto brutta, quasi iilleggibile

Da tradurre:

Magazzino > Prodotti > Tipo intrastat

Per me Ok
+1

AntonioMariaVigliotti

Test superati
Ringrazio lo sviluppatore il lavoro fatto

TheMule71 · 2021-11-19T09:29:48Z

/ocabot merge nobump

OCA-git-bot · 2021-11-19T09:29:58Z

On my way to merge this fine PR!
Prepared branch 14.0-ocabot-merge-pr-2228-by-TheMule71-bump-nobump, awaiting test results.

Signed-off-by TheMule71

OCA-git-bot · 2021-11-19T09:52:02Z

It looks like something changed on 14.0 in the meantime.
Let me try again (no action is required from you).
Prepared branch 14.0-ocabot-merge-pr-2228-by-TheMule71-bump-nobump, awaiting test results.

OCA-git-bot · 2021-11-19T10:15:58Z

Congratulations, your PR was merged at 62424a5. Thanks a lot for contributing to OCA. ❤️

tafaRU · 2021-11-19T14:33:33Z

/ocabot migration l10n_it_intrastat

SimoRubi mentioned this pull request Apr 22, 2021

Migration to version 14.0 #1905

Open

76 tasks

SimoRubi force-pushed the 14.0-mig-l10n_it_intrastat branch from d7be421 to 20fe1d9 Compare April 22, 2021 12:46

TheMule71 added a commit to odoo-italia/l10n-italy that referenced this pull request Apr 23, 2021

added OCA#2225 OCA#2228 OCA#2229 OCA#2230 (was OCA#2199)

440c053

SimoRubi force-pushed the 14.0-mig-l10n_it_intrastat branch 2 times, most recently from 6a764aa to 7efd8ee Compare April 23, 2021 13:53

SimoRubi marked this pull request as ready for review April 26, 2021 09:12

SimoRubi force-pushed the 14.0-mig-l10n_it_intrastat branch from 7efd8ee to 3237a76 Compare April 26, 2021 14:40

fredzamoabg suggested changes Apr 30, 2021

View reviewed changes

fredzamoabg mentioned this pull request Apr 30, 2021

14.0 mig l10n_it_intrastat_statement #2259

Merged

SimoRubi force-pushed the 14.0-mig-l10n_it_intrastat branch from 6825923 to a1e86e1 Compare May 6, 2021 07:59

fredzamoabg approved these changes May 6, 2021

View reviewed changes

TheMule71 added a commit to odoo-italia/l10n-italy that referenced this pull request Jun 6, 2021

added OCA#2225 OCA#2228 OCA#2229 OCA#2230 (was OCA#2199)

4636530

SimoRubi force-pushed the 14.0-mig-l10n_it_intrastat branch from a1e86e1 to 9478485 Compare June 9, 2021 15:33

SimoRubi marked this pull request as draft June 10, 2021 09:24

SimoRubi force-pushed the 14.0-mig-l10n_it_intrastat branch 2 times, most recently from 22770f0 to 813b351 Compare June 14, 2021 14:23

TheMule71 added a commit to odoo-italia/l10n-italy that referenced this pull request Jun 18, 2021

added OCA#2225 OCA#2228 OCA#2229 OCA#2230 (was OCA#2199)

490cc7a

TheMule71 added a commit to odoo-italia/l10n-italy that referenced this pull request Jun 18, 2021

added OCA#2225 OCA#2228 OCA#2229 OCA#2230 (was OCA#2199)

a6d149b

TheMule71 added a commit to odoo-italia/l10n-italy that referenced this pull request Jul 2, 2021

added OCA#2225 OCA#2228 OCA#2229 OCA#2230 (was OCA#2199)

46d6e9c

TheMule71 added a commit to odoo-italia/l10n-italy that referenced this pull request Jul 2, 2021

added OCA#2225 OCA#2228 OCA#2229 OCA#2230 (was OCA#2199)

0aac4fc

TheMule71 added a commit to odoo-italia/l10n-italy that referenced this pull request Jul 9, 2021

added OCA#2225 OCA#2228 OCA#2229 OCA#2230 (was OCA#2199)

0831159

TheMule71 added a commit to odoo-italia/l10n-italy that referenced this pull request Jul 9, 2021

add OCA/web#1952 (pre req for OCA#2228)

5453c41

TheMule71 added a commit to odoo-italia/l10n-italy that referenced this pull request Jul 9, 2021

add OCA/web#1952 (pre req for OCA#2228)

d166896

TheMule71 added a commit to odoo-italia/l10n-italy that referenced this pull request Jul 9, 2021

added OCA#2225 OCA#2228 OCA#2229 OCA#2230 (was OCA#2199)

0d1e9a8

TheMule71 added a commit to odoo-italia/l10n-italy that referenced this pull request Jul 9, 2021

add OCA/web#1952 (pre req for OCA#2228)

6904ca1

TheMule71 added a commit to odoo-italia/l10n-italy that referenced this pull request Jul 9, 2021

tmp fix: OCA#2228 has conflicts during rebase

27b8b34

TheMule71 added a commit to odoo-italia/l10n-italy that referenced this pull request Jul 9, 2021

added OCA#2225 OCA#2228 OCA#2229 OCA#2230 (was OCA#2199)

d59aa06

TheMule71 added a commit to odoo-italia/l10n-italy that referenced this pull request Jul 9, 2021

added OCA#2225 OCA#2228 OCA#2229 OCA#2230 (was OCA#2199)

1be8ba1

l10n_it_intrastat: Move size constraint of fields to web client.

d50e958

Otherwise the following warning is raised: unknown parameter 'size', if this is an actual parameter you may want to override the method _valid_field_parameter on the relevant model in order to allow it

SimoRubi force-pushed the 14.0-mig-l10n_it_intrastat branch from d86a3e3 to d50e958 Compare November 12, 2021 15:35

[REF] l10n_it_intrastat: Cleaner computation of statement section

99f8ec2

SimoRubi requested a review from TheMule71 November 12, 2021 15:57

TheMule71 approved these changes Nov 15, 2021

View reviewed changes

OCA-git-bot added approved ready to merge labels Nov 15, 2021

TheMule71 mentioned this pull request Nov 15, 2021

l10n_it_intrastat: modifica nome variabili #2510

Open

3 tasks

AntonioMariaVigliotti approved these changes Nov 18, 2021

View reviewed changes

OCA-git-bot added a commit that referenced this pull request Nov 19, 2021

Merge PR #2228 into 14.0

126c943

Signed-off-by TheMule71

OCA-git-bot added the bot is merging ⏳ label Nov 19, 2021

OCA-git-bot merged commit bc36628 into OCA:14.0 Nov 19, 2021

OCA-git-bot added merged 🎉 and removed bot is merging ⏳ labels Nov 19, 2021

OCA-git-bot added this to the 14.0 milestone Nov 19, 2021

TheMule71 added a commit to TheMule71/l10n-italy that referenced this pull request Jan 21, 2022

added OCA#2225 OCA#2228 OCA#2229 OCA#2230 (was OCA#2199)

b1228f9

TheMule71 added a commit to TheMule71/l10n-italy that referenced this pull request Jan 21, 2022

added OCA#2225 OCA#2228 OCA#2229 OCA#2230 (was OCA#2199)

6747bba

TheMule71 added a commit to odoo-italia/l10n-italy that referenced this pull request Jun 29, 2022

added OCA#2225 OCA#2228 OCA#2229 OCA#2230 (was OCA#2199)

47a35f7

francesco-ooops added the migration label Jan 10, 2024

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[14.0][MIG] l10n_it_intrastat #2228

[14.0][MIG] l10n_it_intrastat #2228

SimoRubi commented Apr 22, 2021 •

edited

Loading

fredzamoabg Apr 30, 2021

SimoRubi May 6, 2021

TheMule71 commented Apr 30, 2021

SimoRubi commented May 6, 2021

TheMule71 left a comment

OCA-git-bot commented Nov 15, 2021

tafaRU commented Nov 15, 2021

TheMule71 commented Nov 16, 2021

tafaRU commented Nov 18, 2021

AntonioMariaVigliotti commented Nov 18, 2021

AntonioMariaVigliotti left a comment

TheMule71 commented Nov 19, 2021

OCA-git-bot commented Nov 19, 2021

OCA-git-bot commented Nov 19, 2021

OCA-git-bot commented Nov 19, 2021

tafaRU commented Nov 19, 2021

[14.0][MIG] l10n_it_intrastat #2228

[14.0][MIG] l10n_it_intrastat #2228

Conversation

SimoRubi commented Apr 22, 2021 • edited Loading

fredzamoabg Apr 30, 2021

Choose a reason for hiding this comment

SimoRubi May 6, 2021

Choose a reason for hiding this comment

TheMule71 commented Apr 30, 2021

SimoRubi commented May 6, 2021

TheMule71 left a comment

Choose a reason for hiding this comment

OCA-git-bot commented Nov 15, 2021

tafaRU commented Nov 15, 2021

TheMule71 commented Nov 16, 2021

tafaRU commented Nov 18, 2021

AntonioMariaVigliotti commented Nov 18, 2021

AntonioMariaVigliotti left a comment

Choose a reason for hiding this comment

TheMule71 commented Nov 19, 2021

OCA-git-bot commented Nov 19, 2021

OCA-git-bot commented Nov 19, 2021

OCA-git-bot commented Nov 19, 2021

tafaRU commented Nov 19, 2021

SimoRubi commented Apr 22, 2021 •

edited

Loading