[12.0] [IMP] add obliged subject checkbox on partner #1052

primes2h · 2019-02-27T11:00:04Z

Ref. #992 (comment)

Allows to manage future improvements.

l10n_it_fatturapa/views/partner_view.xml

l10n_it_fatturapa/models/partner.py

eLBati

Mancano le etichette dei campi

eLBati · 2019-05-13T13:35:45Z

@primes2h Mi ricordi quali sarebbero i soggetti ma non obbligati a fatturazione elettronica?

primes2h · 2019-05-13T14:26:01Z

Mancano le etichette dei campi

Questa è una cosa strana, in locale compaiono, nel runbot no.
Mi sono dimenticato di segnalarlo.

primes2h · 2019-05-13T14:45:30Z

@primes2h Mi ricordi quali sarebbero i soggetti ma non obbligati a fatturazione elettronica?

Es. le persone fisiche e le associazioni.
Non sono soggetti a fattura elettronica (non hanno alcun obbligo di emissione) ma se sono clienti sei obbligato a emettere fattura elettronica e a inviarla all'SdI.
Devi comunque rilasciare la fattura cartacea, chiamata copia di cortesia (che per loro ha valore legale ai fini fiscali) comunicando che il documento è a disposizione nell'area riservata dell'SdI.

eLBati · 2019-05-13T15:59:16Z

Mi sembra confuso: quindi che significato hanno i campi electronic_invoice_subjected e electronic_invoice_obliged_subject?

Finora electronic_invoice_subjected ha significato "cliente per cui invio fattura elettronica".
Il significato cambierebbe?

primes2h · 2019-05-13T16:06:13Z

Finora electronic_invoice_subjected ha significato "cliente per cui invio fattura elettronica".
Il significato cambierebbe?

È gia cambiato con #992, forse ti conviene rileggere tutta la discussione che ha portato a quella PR (vedi #979).

eLBati · 2019-05-14T08:00:19Z

La #992 conferma ciò che scrivo: electronic_invoice_subjected significa "cliente per cui invio fattura elettronica".

Quindi con questa PR che significato assumerebbero i campi electronic_invoice_subjected e electronic_invoice_obliged_subject?

primes2h · 2019-05-14T08:26:04Z

La #992 conferma ciò che scrivo: electronic_invoice_subjected significa "cliente per cui invio fattura elettronica".

Certo, mi sono spiegato male.
Con "è già cambiato" intendevo rispetto al significato originale che era stato dato a "Soggetto a fattura elettronica".

Quindi con questa PR che significato assumerebbero i campi electronic_invoice_subjected e electronic_invoice_obliged_subject?

Il primo campo rimarrebbe "cliente per cui invio fattura elettronica", in pratica abilita la possibilità di fatturare elettronicamente per quel cliente, visualizzando i pulsanti nella fattura.
Non è necessario che vengano "visualizzati" i campi codice destinatario e pec per il cliente, perché non li possiede non essendo soggetto obbligato, viene usato in automatico il codice predefinito

Il secondo campo si riferisce al significato proprio di "Soggetto obbligato", come categoria definita del cliente.
Un cliente di questo tipo ha l'obbligo di fatturazione elettronica, quindi possiede per forza di cose o un codice destinatario o una PEC, da qui la visualizzazione dei relativi campi selezionando la casella.

primes2h · 2019-07-19T09:24:13Z

Mancano le etichette dei campi

Ho corretto.

primes2h · 2019-07-19T09:57:22Z

Mancano le etichette dei campi

Questa è una cosa strana, in locale compaiono, nel runbot no.

Hai per caso idea del perché può capitare una cosa del genere?

l10n_it_fatturapa/views/partner_view.xml

mrcast · 2019-08-05T13:36:51Z

@eLBati @primes2h quanto si diceva nella #992 mi sembra molto sensato ed è utile portarlo avanti

primes2h · 2019-08-05T15:04:16Z

@eLBati @primes2h quanto si diceva nella #992 mi sembra molto sensato ed è utile portarlo avanti

👍

primes2h · 2019-08-08T19:35:23Z

Adesso dovrebbe essere ok.

labaggio · 2019-12-17T10:26:41Z

@primes2h questa PR è abbandonata da tempo ed ha dei conflitti.. puoi sistemarla, così magari riusciamo a mergiarla?

eLBati · 2019-12-20T07:14:51Z

Mi sono accorto però che nel frattempo è stata implementata la nuova funzionalità di visualizzazione dati di importazione della fattura elettronica del fornitore

@primes2h a che dati ti riferisci?

primes2h · 2019-12-20T15:25:18Z

Mi sono accorto però che nel frattempo è stata implementata la nuova funzionalità di visualizzazione dati di importazione della fattura elettronica del fornitore

@primes2h a che dati ti riferisci?

Errore mio, mi riferivo a questi qui sotto che compaiono quando viene abilitata la fatturazione elettronica su un fornitore. In realtà non è una cosa "nuova", è solo che la modifica relativa a questa PR l'avevo valutata solo lato cliente.

eLBati · 2019-12-23T07:17:33Z

Viene fatto qualche controllo sul codice destinatario/PEC in questo caso?

No:

l10n-italy/l10n_it_fatturapa/models/partner.py

Line 76 in ae853ca

if partner.electronic_invoice_subjected and partner.customer:

A questo punto vi chiedo gentilmente una piccola review funzionale della PR in caso di:

partner che sia solo fornitore

partner sia cliente che fornitore

Come dicevi, se un fornitore è soggetto vuol dire che è italiano e quindi anche obbligato.
Quindi secondo me, per i fornitori, se electronic_invoice_subjected è vero lo deve essere anche electronic_invoice_obliged_subject

primes2h · 2019-12-23T14:14:03Z

Viene fatto qualche controllo sul codice destinatario/PEC in questo caso?

No:

l10n-italy/l10n_it_fatturapa/models/partner.py

Line 76 in ae853ca

if partner.electronic_invoice_subjected and partner.customer:

A questo punto vi chiedo gentilmente una piccola review funzionale della PR in caso di:

partner che sia solo fornitore

partner sia cliente che fornitore

Come dicevi, se un fornitore è soggetto vuol dire che è italiano e quindi anche obbligato.
Quindi secondo me, per i fornitori, se electronic_invoice_subjected è vero lo deve essere anche electronic_invoice_obliged_subject

Si, ha decisamente senso.
Ho implementato la modifica.

primes2h · 2019-12-23T14:30:01Z

Viene fatto qualche controllo sul codice destinatario/PEC in questo caso?

No:

l10n-italy/l10n_it_fatturapa/models/partner.py

Line 76 in ae853ca

if partner.electronic_invoice_subjected and partner.customer:

A questo punto vi chiedo gentilmente una piccola review funzionale della PR in caso di:

partner che sia solo fornitore

partner sia cliente che fornitore

Come dicevi, se un fornitore è soggetto vuol dire che è italiano e quindi anche obbligato.
Quindi secondo me, per i fornitori, se electronic_invoice_subjected è vero lo deve essere anche electronic_invoice_obliged_subject

Si, ha decisamente senso.
Ho implementato la modifica.

Ho anche aggiunto il controllo sul fornitore. Se è straniero, quando viene abilitato electronic_invoice_obliged_subject imposta automaticamente il codice destinatario a XXXXXXX.
Grazie.

eLBati

LGTM

primes2h · 2020-01-20T17:27:05Z

Ho aggiornato il relativo README.

eLBati · 2020-01-22T16:12:58Z

@primes2h il readme viene così
https://github.com/OCA/l10n-italy/blob/6bf4455eec799c31b8f626e16e829542193a20e3/l10n_it_fatturapa/readme/CONFIGURE.rst
è voluto?

primes2h · 2020-01-22T16:18:56Z

@primes2h il readme viene così
https://github.com/OCA/l10n-italy/blob/6bf4455eec799c31b8f626e16e829542193a20e3/l10n_it_fatturapa/readme/CONFIGURE.rst
è voluto?

Mmmhh... no. L'avevo testato in locale con retext ed era ok. Adesso provvedo a sistemarlo.

primes2h · 2020-01-22T16:32:55Z

@primes2h il readme viene così
https://github.com/OCA/l10n-italy/blob/6bf4455eec799c31b8f626e16e829542193a20e3/l10n_it_fatturapa/readme/CONFIGURE.rst
è voluto?

Mmmhh... no. L'avevo testato in locale con retext ed era ok. Adesso provvedo a sistemarlo.

@eLBati adesso dovrebbe essere ok.

eLBati · 2020-01-23T06:34:03Z

/ocabot merge minor

OCA-git-bot · 2020-01-23T06:43:31Z

Hey, thanks for contributing! Proceeding to merge this for you.
Prepared branch 12.0-ocabot-merge-pr-1052-by-eLBati-bump-minor, awaiting test results.

Signed-off-by eLBati

OCA-git-bot · 2020-01-23T07:38:56Z

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

primes2h mentioned this pull request Feb 27, 2019

[12.0] IMP l10n_it_fatturapa better label 'Enable electronic invoicing' to identify partners #992

Merged

eLBati reviewed Feb 28, 2019

View reviewed changes

l10n_it_fatturapa/views/partner_view.xml Show resolved Hide resolved

primes2h force-pushed the 12.0-IMP-l10n_it_fatturapa-partner_e-invoicing_management branch 4 times, most recently from 091f850 to bbf4f2d Compare March 1, 2019 22:03

eLBati changed the title ~~[IMP] add obliged subject checkbox on partner~~ [12.0] [IMP] add obliged subject checkbox on partner Mar 4, 2019

eLBati added the needs review label Mar 4, 2019

primes2h force-pushed the 12.0-IMP-l10n_it_fatturapa-partner_e-invoicing_management branch from bbf4f2d to b33a120 Compare May 8, 2019 12:22

primes2h mentioned this pull request May 8, 2019

[IMP][REF] l10n_it_fatturapa_out: hide 'Export E-invoice' for customers not enabled to e-invoice #1230

Closed

4 tasks

eLBati reviewed May 13, 2019

View reviewed changes

l10n_it_fatturapa/models/partner.py Outdated Show resolved Hide resolved

primes2h force-pushed the 12.0-IMP-l10n_it_fatturapa-partner_e-invoicing_management branch from b33a120 to 9955072 Compare May 13, 2019 12:56

eLBati requested changes May 13, 2019

View reviewed changes

primes2h force-pushed the 12.0-IMP-l10n_it_fatturapa-partner_e-invoicing_management branch from 9955072 to 0427ddf Compare July 19, 2019 08:02

eLBati requested changes Jul 23, 2019

View reviewed changes

l10n_it_fatturapa/views/partner_view.xml Show resolved Hide resolved

l10n_it_fatturapa/views/partner_view.xml Show resolved Hide resolved

primes2h force-pushed the 12.0-IMP-l10n_it_fatturapa-partner_e-invoicing_management branch from 0427ddf to 1a19481 Compare August 8, 2019 19:28

primes2h force-pushed the 12.0-IMP-l10n_it_fatturapa-partner_e-invoicing_management branch from 1a19481 to 69c30b3 Compare December 17, 2019 10:43

primes2h force-pushed the 12.0-IMP-l10n_it_fatturapa-partner_e-invoicing_management branch from 04353bc to 0600b94 Compare December 23, 2019 13:53

primes2h force-pushed the 12.0-IMP-l10n_it_fatturapa-partner_e-invoicing_management branch 2 times, most recently from fa71288 to f6bac7a Compare December 23, 2019 14:26

eLBati approved these changes Jan 3, 2020

View reviewed changes

primes2h force-pushed the 12.0-IMP-l10n_it_fatturapa-partner_e-invoicing_management branch from f6bac7a to c79dc5e Compare January 20, 2020 17:25

primes2h requested a review from eLBati January 20, 2020 17:27

primes2h force-pushed the 12.0-IMP-l10n_it_fatturapa-partner_e-invoicing_management branch 2 times, most recently from a2bc868 to 6bf4455 Compare January 20, 2020 17:30

tafaRU approved these changes Jan 21, 2020

View reviewed changes

OCA-git-bot added the approved label Jan 21, 2020

primes2h force-pushed the 12.0-IMP-l10n_it_fatturapa-partner_e-invoicing_management branch from 6bf4455 to 789228d Compare January 22, 2020 16:28

[IMP] add obliged subject checkbox on partner

5af6cfe

primes2h force-pushed the 12.0-IMP-l10n_it_fatturapa-partner_e-invoicing_management branch from 789228d to 5af6cfe Compare January 22, 2020 16:31

eLBati approved these changes Jan 23, 2020

View reviewed changes

OCA-git-bot added a commit that referenced this pull request Jan 23, 2020

Merge PR #1052 into 12.0

6ccf2fe

Signed-off-by eLBati

OCA-git-bot merged commit 5af6cfe into OCA:12.0 Jan 23, 2020

OCA-git-bot added the merged 🎉 label Jan 23, 2020

primes2h deleted the 12.0-IMP-l10n_it_fatturapa-partner_e-invoicing_management branch December 8, 2020 14:40

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[12.0] [IMP] add obliged subject checkbox on partner #1052

[12.0] [IMP] add obliged subject checkbox on partner #1052

primes2h commented Feb 27, 2019 •

edited

eLBati left a comment

eLBati commented May 13, 2019 •

edited

primes2h commented May 13, 2019

primes2h commented May 13, 2019 •

edited

eLBati commented May 13, 2019

primes2h commented May 13, 2019

eLBati commented May 14, 2019

primes2h commented May 14, 2019

primes2h commented Jul 19, 2019

primes2h commented Jul 19, 2019

mrcast commented Aug 5, 2019

primes2h commented Aug 5, 2019

primes2h commented Aug 8, 2019

labaggio commented Dec 17, 2019

eLBati commented Dec 20, 2019

primes2h commented Dec 20, 2019

eLBati commented Dec 23, 2019

primes2h commented Dec 23, 2019

primes2h commented Dec 23, 2019

eLBati left a comment

primes2h commented Jan 20, 2020

eLBati commented Jan 22, 2020

primes2h commented Jan 22, 2020

primes2h commented Jan 22, 2020

eLBati commented Jan 23, 2020

OCA-git-bot commented Jan 23, 2020

OCA-git-bot commented Jan 23, 2020

[12.0] [IMP] add obliged subject checkbox on partner #1052

[12.0] [IMP] add obliged subject checkbox on partner #1052

Conversation

primes2h commented Feb 27, 2019 • edited

eLBati left a comment

Choose a reason for hiding this comment

eLBati commented May 13, 2019 • edited

primes2h commented May 13, 2019

primes2h commented May 13, 2019 • edited

eLBati commented May 13, 2019

primes2h commented May 13, 2019

eLBati commented May 14, 2019

primes2h commented May 14, 2019

primes2h commented Jul 19, 2019

primes2h commented Jul 19, 2019

mrcast commented Aug 5, 2019

primes2h commented Aug 5, 2019

primes2h commented Aug 8, 2019

labaggio commented Dec 17, 2019

eLBati commented Dec 20, 2019

primes2h commented Dec 20, 2019

eLBati commented Dec 23, 2019

primes2h commented Dec 23, 2019

primes2h commented Dec 23, 2019

eLBati left a comment

Choose a reason for hiding this comment

primes2h commented Jan 20, 2020

eLBati commented Jan 22, 2020

primes2h commented Jan 22, 2020

primes2h commented Jan 22, 2020

eLBati commented Jan 23, 2020

OCA-git-bot commented Jan 23, 2020

OCA-git-bot commented Jan 23, 2020

primes2h commented Feb 27, 2019 •

edited

eLBati commented May 13, 2019 •

edited

primes2h commented May 13, 2019 •

edited