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.

Sign up for GitHub

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

Jump to bottom

[10.0] porting l10n_it_fatturapa_in and supporting B2B (in and out) #589

Merged
merged 5 commits into from
Oct 11, 2018
Merged

[10.0] porting l10n_it_fatturapa_in and supporting B2B (in and out) #589

merged 5 commits into from
Oct 11, 2018

Conversation

eLBati
Copy link
Member

@eLBati eLBati commented Sep 5, 2018 •
edited
Loading

Dipende da

TODO list

@eLBati eLBati added this to the 10.0 milestone Sep 5, 2018
@eLBati eLBati mentioned this pull request Sep 5, 2018
Closed
@eLBati eLBati force-pushed the 10.0-porting-fatturapa_in branch 3 times, most recently from 57648a5 to 335743e Compare September 5, 2018 14:46
@eLBati eLBati changed the title 10.0 porting fatturapa in [10.0] porting l10n_it_fatturapa_in and supporting B2B (in and out) Sep 8, 2018
@eLBati eLBati force-pushed the 10.0-porting-fatturapa_in branch from d82810e to 42725d1 Compare September 10, 2018 16:24
@eLBati
copying l10n_it_fatturapa_in from 7.0 as is
7dc651d
@eLBati
porting
cb063e9 
l10n_it_fatturapa_in to 10
(REF withholding tax integration, invoice interface, fiscal document type
ADD some DatiGeneraliDocumento fields)
ADD _setTerzoIntermediarioOSoggettoEmittente to l10n_it_fatturapa_out
@eLBati eLBati force-pushed the 10.0-porting-fatturapa_in branch 8 times, most recently from 4dd9124 to 7a8d04a Compare September 17, 2018 10:29
@eLBati eLBati force-pushed the 10.0-porting-fatturapa_in branch 6 times, most recently from c0b8d9e to 7d07191 Compare September 24, 2018 16:26
@eLBati eLBati force-pushed the 10.0-porting-fatturapa_in branch 5 times, most recently from b0e7e0b to 25427f0 Compare October 2, 2018 15:44
@eLBati eLBati force-pushed the 10.0-porting-fatturapa_in branch from 25427f0 to 3dd8ff9 Compare October 5, 2018 07:49
@baba75
Copy link

baba75 commented Oct 5, 2018

Il campo per il numero della fattura non può essere più lungo di 10 caratteri, per cui se ho la sequenza standard FAT/%(range_years)/ ottengo un errore di validazione

@eLBati eLBati force-pushed the 10.0-porting-fatturapa_in branch from 3dd8ff9 to a4c1ff5 Compare October 6, 2018 07:58
@eLBati
Copy link
Member Author

eLBati commented Oct 6, 2018

@baba75 non mi risulta. In questo file di test
https://github.com/eLBati/l10n-italy/blob/10.0-porting-fatturapa_in/l10n_it_fatturapa_out/tests/data/IT06363391001_00006.xml#L67
il numero fattura è INV/2018/0013, lungo 13.
Tale XML rispetta lo schema ed il test del SDI viene superato

@eLBati eLBati force-pushed the 10.0-porting-fatturapa_in branch 2 times, most recently from f562be3 to 5ee52c2 Compare October 7, 2018 05:30
@eLBati eLBati mentioned this pull request Oct 7, 2018
Merged
@eLBati eLBati force-pushed the 10.0-porting-fatturapa_in branch from 5ee52c2 to eb37c65 Compare October 7, 2018 07:01
@baba75
Copy link

baba75 commented Oct 8, 2018

@eLBati forse interpreto male il messaggio di errore. Se lascio la sequenza standard mi esce questo:

FatturaElettronicaHeader.DatiTrasmissione.ProgressivoInvio:
Type {http://ivaservizi.agenziaentrate.gov.it/docs/xsd/fatture/v1.2}String10Type pattern constraint violated by value FAT/2018/0009

Se cambio la sequenza in 18/0009 per esempio, funziona tutto bene.

@eLBati eLBati force-pushed the 10.0-porting-fatturapa_in branch 4 times, most recently from 121c234 to caf93ab Compare October 8, 2018 09:43
@eLBati
Copy link
Member Author

eLBati commented Oct 8, 2018 •
edited
Loading

@baba75 probabilmente hai usato la sequence di numerazione delle fatture anche per la numerazione del file.
Per la numerazione del file bisogna usarne una specifica, ad esempio quella proposta dal modulo:
https://github.com/eLBati/l10n-italy/blob/10.0-porting-fatturapa_in/l10n_it_fatturapa/data/fatturapa_data.xml#L109

Usare quella delle fatture, tra l'altro, comporta anche dei buchi nella numerazione delle fatture

Ho fatto in modo che tale sequence venga automaticamente valorizzata di default:
https://github.com/eLBati/l10n-italy/blob/10.0-porting-fatturapa_in/l10n_it_fatturapa/models/company.py#L126

@eLBati eLBati force-pushed the 10.0-porting-fatturapa_in branch 3 times, most recently from 698b6c1 to fc04995 Compare October 8, 2018 13:23
eLBati added 2 commits October 8, 2018 17:36
@eLBati
l10n_it_ipa ADD is_pa bool field
f3652eb 
ADD PECDestinatario, CodiceDestinatario , FormatoTrasmissione handling
ADD Lotto di fatture verso soggetto privato
ADD gestione prodotti nelle fatture passive
ADD l10n_it_fatturapa_in_purchase
ADD einvoice.line model
IMP form fatture
ADD Livello di dettaglio Fatture elettroniche
ADD PDF preview dell'XML
ADD Link to existing supplier invoice
and handle "registered" XML
ADD rappresentante fiscale e stabile organizzazione in emissione
gestione prodotti DatiBollo , DatiCassaPrevidenziale e ScontoMaggiorazione
CodiceArticolo, ftpa_line_number

IMP "show preview" as link
FIX invoice duplication

ADD constraints and Nome Cognome

Avoid to cancel invoice with XML

Mostra i dati fattura elettronica solo se il cliente è soggetto
IMP READMEs

Gestione IVA inclusa nel prezzo
@eLBati
ADD l10n_it_fatturapa_out_ddt
5f1ab23 
IMP PEP8
@eLBati eLBati force-pushed the 10.0-porting-fatturapa_in branch 3 times, most recently from a9e828e to 23452c4 Compare October 9, 2018 12:15
@eLBati
IMP decimal precision
3054766 
ADD Export E-invoice button
IMP README
@eLBati eLBati force-pushed the 10.0-porting-fatturapa_in branch from 23452c4 to 3054766 Compare October 9, 2018 12:29
@eLBati
Copy link
Member Author

eLBati commented Oct 9, 2018

@warp10 @sergiocorato @alessandrocamilli @As400it @baba75 merge?

@sergiocorato
Copy link
Contributor

per me ok

@baba75
Copy link

baba75 commented Oct 9, 2018

Ok anche per me

@eLBati eLBati merged commit a07d9c7 into OCA:10.0 Oct 11, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@sergiocorato sergiocorato sergiocorato left review comments

Assignees
No one assigned
Labels
needs review
Projects
None yet
Milestone
10.0
Development

Successfully merging this pull request may close these issues.
3 participants
@eLBati @baba75 @sergiocorato