Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[14.0][MIG] l10n_it_pos_fatturapa migration to v14 #2739

Closed
wants to merge 11 commits into from

Conversation

robyf70
Copy link
Contributor

@robyf70 robyf70 commented Mar 30, 2022

No description provided.

@robyf70 robyf70 changed the title [14.0 mig l10n it pos fatturapa [14.0][MIG] l10n_it_pos_fatturapa migration to v14 Mar 30, 2022
@robyf70 robyf70 force-pushed the 14.0-mig-l10n_it_pos_fatturapa branch 2 times, most recently from 736e86b to 6ad7137 Compare April 1, 2022 12:46
@robyf70 robyf70 force-pushed the 14.0-mig-l10n_it_pos_fatturapa branch from 6ad7137 to ccd6098 Compare April 13, 2022 07:42
@robyf70
Copy link
Contributor Author

robyf70 commented Apr 13, 2022

@eLBati @francesco-ooops Potete fare un review anche se ho visto che fallisce a causa di ERROR: TestReverseCharge.test_intra_EU_zero_total?

@francesco-ooops
Copy link
Contributor

@robyf70 crea nuovo cliente > abilita e-fattura > mancano i controlli CF/P.IVA e indirizzo al salvataggio

@robyf70
Copy link
Contributor Author

robyf70 commented Apr 13, 2022

@robyf70 crea nuovo cliente > abilita e-fattura > mancano i controlli CF/P.IVA e indirizzo al salvataggio

Non ho capito che tipo di controlli intendi. Che sia valorizzato CF o P.IVA?

@francesco-ooops
Copy link
Contributor

@robyf70 crea nuovo cliente > abilita e-fattura > mancano i controlli CF/P.IVA e indirizzo al salvataggio

Non ho capito che tipo di controlli intendi. Che sia valorizzato CF o P.IVA?

esatto, stessa cosa per per indirizzo, come avviene sulla 12 e come avviene nel modello del contatto

@robyf70
Copy link
Contributor Author

robyf70 commented Apr 13, 2022

@robyf70 crea nuovo cliente > abilita e-fattura > mancano i controlli CF/P.IVA e indirizzo al salvataggio

Non ho capito che tipo di controlli intendi. Che sia valorizzato CF o P.IVA?

esatto, stessa cosa per per indirizzo, come avviene sulla 12 e come avviene nel modello del contatto

Ho controllato la 12, ma nel codice non c'è questo tipo di controllo

@francesco-ooops
Copy link
Contributor

@robyf70 crea nuovo cliente > abilita e-fattura > mancano i controlli CF/P.IVA e indirizzo al salvataggio

Non ho capito che tipo di controlli intendi. Che sia valorizzato CF o P.IVA?

esatto, stessa cosa per per indirizzo, come avviene sulla 12 e come avviene nel modello del contatto

Ho controllato la 12, ma nel codice non c'è questo tipo di controllo

Quello che posso testare è che se creo un contatto e abilito il flag fattura elettronica nel modello contatti devo inserire CF/P.IVA e indirizzo, mentre nel POS questo vincolo non c'è: https://recordit.co/fnp8TcFxet

@robyf70
Copy link
Contributor Author

robyf70 commented Apr 13, 2022

Tra l'altro pensavo di aggiungere il check di validazione della partita iva

@francesco-ooops
Copy link
Contributor

@robyf70 da quale modulo viene la funzione di validazione delle P.IVA? ho notato che è attiva anche se il flag "Verifica delle partite IVA con il servizio europeo VIES" è disattivato

@robyf70
Copy link
Contributor Author

robyf70 commented Apr 13, 2022

@robyf70 da quale modulo viene la funzione di validazione delle P.IVA? ho notato che è attiva anche se il flag "Verifica delle partite IVA con il servizio europeo VIES" è disattivato

Dovrebbe essere un modulo base

@robyf70
Copy link
Contributor Author

robyf70 commented Apr 13, 2022

quello VIES è un'estensione dello standard.

Copy link
Contributor

@francesco-ooops francesco-ooops left a comment

Choose a reason for hiding this comment

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

questa PR sulla sx, v.12 sulla dx: https://recordit.co/nsI10CjCDl

creo nuovo cliente > abilito e-fattura > salvo

in v.12, se non c'è P.IVA o CF, ricevo messaggio di errore
in questa PR no

@francesco-ooops
Copy link
Contributor

@robyf70 riesci a metterci le mani a breve?

@robyf70
Copy link
Contributor Author

robyf70 commented Apr 26, 2022

@robyf70 riesci a metterci le mani a breve?

ho una modifica che fà il controllo della partita iva direttamente da JS, vedo se nel pomeriggio riesco a rifare i tutti i check

@TheMule71
Copy link
Contributor

Ho controllato la 12, ma nel codice non c'è questo tipo di controllo

Quello che posso testare è che se creo un contatto e abilito il flag fattura elettronica nel modello contatti devo inserire CF/P.IVA e indirizzo, mentre nel POS questo vincolo non c'è: https://recordit.co/fnp8TcFxet

Credo che il codice sia questo:

if (
not partner.vat and not partner.fiscalcode and
partner.country_id.code == 'IT'
):
raise ValidationError(_(
"Italian partner %s must "
"have VAT Number or Fiscal Code."
) % partner.name)

@francesco-ooops
Copy link
Contributor

@robyf70 ok per te se portiamo avanti il lavoro fatto fin qui?

@robyf70
Copy link
Contributor Author

robyf70 commented May 19, 2022

@robyf70 ok per te se portiamo avanti il lavoro fatto fin qui?

Per me ok

@francesco-ooops
Copy link
Contributor

@OCA/local-italy-maintainers si può chiudere

@tafaRU
Copy link
Member

tafaRU commented Jul 14, 2023

Chiudo in favore di #3482

@tafaRU tafaRU closed this Jul 14, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants