DinamicheAziendali / l10n_it_delivery_note

module to create DDT for Italian market
GNU Affero General Public License v3.0
2 stars 5 forks source link

Notifica di DdT non validato a livello di azienda? #71

Closed Byloth closed 3 years ago

Byloth commented 3 years ago

Buongiorno a tutti.

Richiamo l'attenzione di @As400it.

Stavo girovagando tra i file del progetto e mi son ritrovato di fronte a questa incongruenza.
Per quale motivo, abbiamo definito questi due flag di configurazioni uno a livello di "ruolo utente" mentre l'altro a livello di "configurazione azienda"?

C'era un motivo in particolare?

MarcoCalcagni commented 3 years ago

per disciplinare se abilitare o meno la fatturazione dal delivery sia a livello azienda che a livello utente nel senso che se fatturo dai ddt potrei non abilitare il magazziniere ad emettere fattura.

Il giorno lun 12 ott 2020 alle ore 18:00 Matteo Bilotta < notifications@github.com> ha scritto:

Buongiorno a tutti.

Richiamo l'attenzione di @As400it https://github.com/As400it.

Stavo girovagando tra i file del progetto e mi son ritrovato di fronte a questa incongruenza. Per quale motivo, abbiamo definito questi due flag https://github.com/DinamicheAziendali/l10n_it_delivery_note/blob/98545e5f50b548234dd266fbc028edc61d179f00/l10n_it_delivery_note/models/res_config_settings.py di configurazioni uno a livello di "ruolo utente" mentre l'altro a livello di "configurazione azienda"?

C'era un motivo in particolare?

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/DinamicheAziendali/l10n_it_delivery_note/issues/71, or unsubscribe https://github.com/notifications/unsubscribe-auth/ACUCM3Q5S5XX7SVK4WD44A3SKMRZLANCNFSM4SM7KFFQ .

-- Dott. Marco Calcagni

Dinamiche Aziendali S.r.l. Sede Legale: Via G.Porzio, 4 - 80143 Centro Direzionale di Napoli Sede Operativa: Via Nazionale delle puglie, 3 - 80013 Casalnuovo di Napoli (NA) Codice Fiscale e Partita IVA 07655230634 Tel +39 081 842 55 11 Cellulare +39 329 15 44 306 Fax +39 081 00 99 503 pec: dinamicheaziendali@pec.it www.dinamicheaziendali.it


Nota di riservatezza / Confidentiality Notice:

Il messaggio può contenere informazioni strettamente confidenziali, urgenti e coperte da segreto professionale. Salvo che non sia espressamente autorizzato è, pertanto, strettamente proibito qualsiasi uso, comunicazione, copia o diffusione non autorizzata dei contenuti di questo documento. Nel caso in cui abbiate ricevuto per errore la presente comunicazione, vogliate cortesemente darcene immediata notizia e cancellare questo messaggio dal Vostro sistema. In ogni caso, il mittente, in ragione del mezzo di trasmissione utilizzato, non assume alcuna responsabilità in merito alla segretezza e riservatezza delle informazioni contenute nel presente messaggio e nei relativi allegati.

This message may contain information which are strictly confidential, urgent and protected by professional secrecy. It is then strictly prohibited, unless expressly authorized, any use, disclosure, copying or distribution of the content of this communication. If you have received this communication in error, please notify us immediately and then delete it from your system. In any case, considering the means of transmission, we do not undertake any liability with respect to the secrecy and confidentiality of the information contained in this e-mail and in its attachments.

Byloth commented 3 years ago

@As400it: ho compreso quanto hai descritto...
Il mio dubbio, più che altro, era riferito all'altro campo; quello di notifica durante la fatturazione di un SO con dei DdT in bozza.

Ho creato una PR (#74) in cui cambio questo comportamento da settaggio sulla company a, invece, un ruolo utente (configurabile, eventualmente, anche globalmente)...

È una modifica corretta, lecita e si può mergeare oppure lasciamo così com'è questa logica e non mergeamo la PR?

Byloth commented 3 years ago

Confrontandoci con @As400it, siamo giunti alla conclusione che questa configurazione risulta essere inutile.
Infatti, il tipo di notifica mostrata, dovrà essere visualizzata sempre ed in ogni caso.

Verrà rimossa questa configurazione e modificate le logiche di conseguenza.