[12.0][ADD] l10n_it_sale_commission_riba #3204

odooNextev · 2023-03-07T07:46:01Z

Il modulo permette di integrare il pagamento riba con le commissioni agenti.
Una fattura con termini di pagamento Riba ed emissione con tipologia "Salvo buon fine" verrà presa in considerazione nella generazione delle provvigioni agente solamente trascorsi 5 giorni dall'effettiva scadenza.
E' possibile inserire aggiungere "Senza commissioni" ad una fattura, in modo che non vengano generate commissioni agente.

elvise · 2023-03-07T08:18:15Z

@odooNextev grazie della PR risulterà molto utile!
Ho alcune domande:
1 Il valore 5 giorni è pensato come configurabile dall’utente ?
2 Come si comporta il modulo in caso di Ri.Ba. insoluta ?

odooNextev · 2023-03-07T08:25:17Z

@odooNextev grazie della PR risulterà molto utile! Ho alcune domande: 1 Il valore 5 giorni è pensato come configurabile dall’utente ? 2 Come si comporta il modulo in caso di Ri.Ba. insoluta ?

Siamo contenti che possa tornare utile a qualcuno :)

Per ora no, è fisso perchè i nostri clienti avevano sempre quel margine, però si potrebbe aggiungere come parametro
In caso di riba insoluta le commissioni non vengono incluse nella liquidazione, i nostri clienti volevano questo comportamento

elvise · 2023-03-07T08:35:05Z

@odooNextev grazie della risposta!
Per il punto numero uno pensi che sia qualcosa che puoi introdurre già da questa PR ?
Per il secondo punto forse è bene seguire lo “standard” del modulo commission, all’interno del tipo di commissioni c’è la configurazione “Invoice Status” con due opzioni:
Invoiced Based
Payment Based

Cosa ne pensi ?

stenext · 2023-03-07T08:57:41Z

@odooNextev grazie della risposta! Per il punto numero uno pensi che sia qualcosa che puoi introdurre già da questa PR ? Per il secondo punto forse è bene seguire lo “standard” del modulo commission, all’interno del tipo di commissioni c’è la configurazione “Invoice Status” con due opzioni: Invoiced Based Payment Based

Cosa ne pensi ?

Va bene, proviamo a fare entrambe le modifiche subito

elvise · 2023-03-07T09:03:51Z

@odooNextev grazie della risposta! Per il punto numero uno pensi che sia qualcosa che puoi introdurre già da questa PR ? Per il secondo punto forse è bene seguire lo “standard” del modulo commission, all’interno del tipo di commissioni c’è la configurazione “Invoice Status” con due opzioni: Invoiced Based Payment Based
Cosa ne pensi ?

Va bene, proviamo a fare entrambe le modifiche subito

Fantastico!

odooNextev · 2023-03-07T09:14:44Z

Andrebbe bene aggiungere un campo all'interno della configurazione Ri.Ba dove inserire i giorni da tenere in considerazione?

elvise · 2023-03-07T09:22:31Z

Andrebbe bene aggiungere un campo all'interno della configurazione Ri.Ba dove inserire i giorni da tenere in considerazione?

Direi ottimo! 💣

francesco-ooops · 2023-03-07T09:40:25Z

@odooNextev dato che ci state lavorando, qualche modifica formale (sicuramente ce ne saranno altre):

rimuovere il file .pot che viene generato automaticamente
spostare i contenuti del readme nei files DESCRIPTION e USAGE nella cartella readme
sistemare name, summary e traduzione in inglese come da linee guida

Se serve potete prendere come riferimento la PR #3155 che è stata appena tirata a lucido 😬

forse il nome del modulo dovrebbe essere l10n_it_ricevute_bancarie_commission, @primes2h che dici?

stenext · 2023-03-07T10:42:18Z

@odooNextev dato che ci state lavorando, qualche modifica formale (sicuramente ce ne saranno altre):

rimuovere il file .pot che viene generato automaticamente

spostare i contenuti del readme nei files DESCRIPTION e USAGE nella cartella readme

sistemare name, summary e traduzione in inglese come da linee guida

Se serve potete prendere come riferimento la PR #3155 che è stata appena tirata a lucido 😬

forse il nome del modulo dovrebbe essere l10n_it_ricevute_bancarie_commission, @primes2h che dici?

Abbiamo guardato la PR però troviamo solo DESCRIPTION e USAGE, mentre contributors, mantainers, ecc dove si mettono?
Nono trovo una guida per redigere i template e non mi funziona nemmeno questo: https://github.com/OCA/maintainer-tools/blob/master/README.md#readme-generator

Ho lanciato questo comando che dovrebbe generare tutta la struttura, ma esce senza aver fatto né segnalato niente:
oca-gen-addon-readme --repo-name=l10n-italy --branch=12.0-mig-l10n_otaly_riba_commissions --addon-dir=l10n_it_sale_commission_riba

francesco-ooops · 2023-03-07T11:08:14Z

@stenext questa vista? https://github.com/OCA/maintainer-tools/blob/master/template/module/README.rst

stenext · 2023-03-07T11:13:38Z

@stenext questa vista? https://github.com/OCA/maintainer-tools/blob/master/template/module/README.rst

ok, grazie, però senza il readme.rst nella root del modulo non vengono presi in considerazione i file nella cartella readme
ora infatti abbiamo risolto i problemi che segnalavi nel commento prima con un commit, ma spostando i contenuti nella cartella readme, non viene visualizzato nulla aprendo il modulo: https://github.com/odooNextev/l10n-italy/tree/12.0-mig-l10n_otaly_riba_commissions/l10n_it_sale_commission_riba

francesco-ooops · 2023-03-07T11:27:12Z

@stenext puoi lasciarlo vuoto, eg: https://github.com/OCA/commission/pull/386/files#diff-f7c04cfc2dd7f98c26a7ab5ee6515e4b9b3ad2e588c9b1ec049e0f2d71196602

le info del readme vanno su manifest e files nella cartella readme

stenext · 2023-03-07T13:18:37Z

@stenext puoi lasciarlo vuoto, eg: https://github.com/OCA/commission/pull/386/files#diff-f7c04cfc2dd7f98c26a7ab5ee6515e4b9b3ad2e588c9b1ec049e0f2d71196602

le info del readme vanno su manifest e files nella cartella readme

Ok @francesco-ooops, l'ho fatto però poi navigando nel sorgente non compare più come nella maggior parte dei repo OCA
Inoltre anche nella pagina del modulo nelle app di Odoo non viene mostrato quello che c'è scritto nella cartella readme

tafaRU · 2023-03-07T13:35:43Z

@odooNextev ciò che segnali è normale. Considera che il file README viene generato automaticamente al merge delle PR così come indicato in https://github.com/OCA/maintainer-tools/blob/master/template/module/README.rst
Nel caso tu voglia generarlo in autonomia puoi seguire le indicazioni contenute in https://github.com/OCA/maintainer-tools#readme-generator

francesco-ooops · 2023-03-07T13:41:07Z

@stenext sarà visibile dopo il merge

stenext · 2023-03-07T13:45:10Z

@tafaRU grazie Alex, era lo stesso file che mi aveva indicato prima @francesco-ooops, ma non avevo capito subito fino in fondo...
Scusate, ma è la prima volta che pubblichiamo un modulo completamente nostro su OCA ed alcune info non sono immediate da trovare.
Tra l'altro volevo usare i comandi di oca-mantainers-tools, ma escono tutti senza errori, ma senza fare altro.
Penso sia qualcosa nel venv perchè cercando di generare il token restituisce questo:
AttributeError: module 'github3' has no attribute 'authorize'
Non vorrei andare OT, ma magari voi li usate

stenext · 2023-03-07T14:17:45Z

Comunque @matteonext sta iniziando a creare oggetti e fare test nel runboat http://oca-l10n-italy-12-0-pr3204-2ec199fc6540.runboat.odoo-community.org/web per facilitare eventuali benefattori che vogliano fare una review ;)

primes2h · 2023-03-07T15:19:10Z

Nono trovo una guida per redigere i template e non mi funziona nemmeno questo: https://github.com/OCA/maintainer-tools/blob/master/README.md#readme-generator

Ho lanciato questo comando che dovrebbe generare tutta la struttura, ma esce senza aver fatto né segnalato niente: oca-gen-addon-readme --repo-name=l10n-italy --branch=12.0-mig-l10n_otaly_riba_commissions --addon-dir=l10n_it_sale_commission_riba

Il branch si riferisce alla versione per la quale stai generando il README. In questo caso quindi devi mettere --branch=12.0.
In --addon-dir non devi solo mettere il nome della directory ma devi anche fornire il percorso (relativo o completo). Altrimenti non sa dove cercare.

Anche se il README viene creato in modo automatico dopo il merge, in caso di moduli nuovi può essere comodo e utile generarlo lo stesso.
Consente ai nuovi di prendere mano con gli strumenti messi a disposizione di OCA e ai revisori di valutare a colpo d'occhio la resa finale del testo formattato. Può capitare che alcune cose sfuggano guardando il testo non formattato, in markdown anche una indentazione può fare la differenza per il risultato finale.

stenext · 2023-03-07T15:29:46Z

Grazie @primes2h!
Con le 2 correzioni che mi hai detto il comando funziona.
Lo riaggiungo allora

primes2h · 2023-03-07T15:41:33Z

@odooNextev dato che ci state lavorando, qualche modifica formale (sicuramente ce ne saranno altre):
* rimuovere il file .pot che viene generato automaticamente

In generale non è necessario che il .pot venga rimosso.
Se è stato generato correttamente e/o aggiornato in conseguenza di eventuali modifiche a codice/stringhe si può tranquillamente lasciare.

stefano-ooops

testato funziona

francesco-ooops · 2023-03-09T09:16:41Z

@matteonext una volta testato lascia pure una review, grazie!

matteonext

testato, funziona correttamente

francesco-ooops · 2023-03-09T09:57:59Z

@OCA/local-italy-maintainers qualcuno può fare review? grazie :)

francesco-ooops · 2023-03-13T08:38:22Z

@odooNextev potete squashare i commits?

stenext · 2023-03-13T10:39:48Z

@odooNextev potete squashare i commits?

@francesco-ooops Fatto

francesco-ooops · 2023-03-13T10:44:16Z

@stenext ok, questa è da fixare: https://github.com/OCA/odoo-community.org/blob/master/website/Contribution/CONTRIBUTING.rst#commit-message

@primes2h altre indicazioni formali per avvicinarci al merge? grazie!

… country

Extract commercial partner retrieval Extract partner search Use the actual domain from multi-company partners sharing rule

Only CedentePrestatore and CessionarioCommittente are checked for VAT Groups, when other partner fields match multiple partners just log an inconsistency

Updated by "Update PO files to match POT (msgmerge)" hook in Weblate. Translation: l10n-italy-12.0/l10n-italy-12.0-l10n_it_fatturapa_in Translate-URL: https://translation.odoo-community.org/projects/l10n-italy-12-0/l10n-italy-12-0-l10n_it_fatturapa_in/

…ginal file content

Updated by "Update PO files to match POT (msgmerge)" hook in Weblate. Translation: l10n-italy-12.0/l10n-italy-12.0-l10n_it_fatturapa_pec Translate-URL: https://translation.odoo-community.org/projects/l10n-italy-12-0/l10n-italy-12-0-l10n_it_fatturapa_pec/

Currently translated at 73.9% (352 of 476 strings) Translation: l10n-italy-12.0/l10n-italy-12.0-assets_management Translate-URL: https://translation.odoo-community.org/projects/l10n-italy-12-0/l10n-italy-12-0-assets_management/it/

… alla modifica di sale_commission per filtrare le righe degli agenti

odooNextev · 2023-03-22T09:08:59Z

Chiudo perchè sostituita da: #3222

primes2h · 2023-03-24T09:53:52Z

@odooNextev dato che ci state lavorando, qualche modifica formale (sicuramente ce ne saranno altre):

Se serve potete prendere come riferimento la PR #3155 che è stata appena tirata a lucido grimacing

forse il nome del modulo dovrebbe essere l10n_it_ricevute_bancarie_commission, @primes2h che dici?

Eccomi, perdona il ritardo nella risposta. [*]
Invertire l'ordine mi sembra corretto visto che siamo nel repository l10n-italy.
Come nome suggerisco l10n_it_riba_sale_commission per diverse ragioni:

meglio usare il nome del modulo da cui dipende (quindi _sale_commission e non solo _commission)
meglio usare l'acronimo _riba_, permette di mantenere pù compatto il nome del modulo che altrimente verrebbe molto lungo. Questo si lega anche ad altre modifiche che vorrei suggerire a breve nella PR di migrazione alla 16.0 del modulo l10n_it_ricevute_bancarie. Andavano fatte nella migrazione alla 14.0 ma me la sono persa.

[*] anche se la PR è chiusa intanto rispondo qui, poi commenterò su quella nuova quando sarà pronta per la revisione.

stefano-ooops approved these changes Mar 9, 2023

View reviewed changes

This comment was marked as resolved.

Sign in to view

matteonext approved these changes Mar 9, 2023

View reviewed changes

OCA-git-bot added the approved label Mar 9, 2023

odooNextev force-pushed the 12.0-mig-l10n_otaly_riba_commissions branch from 4a03965 to d75442f Compare March 13, 2023 09:57

michelerusti and others added 21 commits March 21, 2023 17:23

[12.0][FIX] l10n_it_intrastat_statement | report did not show correct…

c7cde80

… country

l10n_it_intrastat_statement 12.0.1.4.1

1a7b147

[UPD] addons table in README.md

be7bf7b

[COV] l10n_it_fatturapa_in: Error when there are multiple partners

bce5796

[REF] l10n_it_fatturapa_in: Include partners in error message

c530b62

Extract commercial partner retrieval Extract partner search Use the actual domain from multi-company partners sharing rule

[FIX] l10n_it_fatturapa_in: Manage VAT groups

39af2df

[IMP] l10n_it_fatturapa_in: Skip Duplicated Partners for some fields

d73bc0c

Only CedentePrestatore and CessionarioCommittente are checked for VAT Groups, when other partner fields match multiple partners just log an inconsistency

[UPD] Update l10n_it_fatturapa_in.pot

de7f8ac

l10n_it_fatturapa_in 12.0.3.1.4

87242c3

[UPD] addons table in README.md

0272929

Update translation files

b978164

Updated by "Update PO files to match POT (msgmerge)" hook in Weblate. Translation: l10n-italy-12.0/l10n-italy-12.0-l10n_it_fatturapa_in Translate-URL: https://translation.odoo-community.org/projects/l10n-italy-12-0/l10n-italy-12-0-l10n_it_fatturapa_in/

[FIX] l10n_it_fatturapa_pec: Receive base64 attachments

90f8d09

[REF] l10n_it_sdi_channel: Documentation, receive_fe receives the ori…

c2cfd2b

…ginal file content

[UPD] Update l10n_it_fatturapa_pec.pot

f4529a3

l10n_it_fatturapa_pec 12.0.2.1.1

da0bf18

l10n_it_sdi_channel 12.0.2.2.2

a2f757a

[UPD] addons table in README.md

cb5ab21

Update translation files

bd49daa

Updated by "Update PO files to match POT (msgmerge)" hook in Weblate. Translation: l10n-italy-12.0/l10n-italy-12.0-l10n_it_fatturapa_pec Translate-URL: https://translation.odoo-community.org/projects/l10n-italy-12-0/l10n-italy-12-0-l10n_it_fatturapa_pec/

Translated using Weblate (Italian)

9a93cc5

Currently translated at 73.9% (352 of 476 strings) Translation: l10n-italy-12.0/l10n-italy-12.0-assets_management Translate-URL: https://translation.odoo-community.org/projects/l10n-italy-12-0/l10n-italy-12-0-assets_management/it/

Rimozione data inizio generazione provvigioni

1af9fb8

[ADD] Aggiunto modulo l10n_it_sale_commission_riba

21c9c78

odooNextev force-pushed the 12.0-mig-l10n_otaly_riba_commissions branch 2 times, most recently from 73ef9f8 to 21c9c78 Compare March 22, 2023 07:36

[IMP] Modifica alla modulo l10n_it_sale_commission_riba per adeguarsi…

804e559

… alla modifica di sale_commission per filtrare le righe degli agenti

odooNextev closed this Mar 22, 2023

primes2h mentioned this pull request Mar 24, 2023

[14.0][MIG] l10n it asset history import #3227

Closed

primes2h mentioned this pull request Apr 27, 2023

[14.0][ADD] l10n_it_riba_sale_commission #3230

Merged

odooNextev deleted the 12.0-mig-l10n_otaly_riba_commissions branch June 1, 2023 13:09

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[12.0][ADD] l10n_it_sale_commission_riba #3204

[12.0][ADD] l10n_it_sale_commission_riba #3204

odooNextev commented Mar 7, 2023

elvise commented Mar 7, 2023

odooNextev commented Mar 7, 2023

elvise commented Mar 7, 2023

stenext commented Mar 7, 2023

elvise commented Mar 7, 2023

odooNextev commented Mar 7, 2023

elvise commented Mar 7, 2023

francesco-ooops commented Mar 7, 2023

stenext commented Mar 7, 2023

francesco-ooops commented Mar 7, 2023

stenext commented Mar 7, 2023

francesco-ooops commented Mar 7, 2023

stenext commented Mar 7, 2023

tafaRU commented Mar 7, 2023

francesco-ooops commented Mar 7, 2023

stenext commented Mar 7, 2023

stenext commented Mar 7, 2023

primes2h commented Mar 7, 2023

stenext commented Mar 7, 2023

primes2h commented Mar 7, 2023

stefano-ooops left a comment

francesco-ooops commented Mar 9, 2023

This comment was marked as resolved.

matteonext left a comment

francesco-ooops commented Mar 9, 2023

francesco-ooops commented Mar 13, 2023

stenext commented Mar 13, 2023 •

edited

Loading

francesco-ooops commented Mar 13, 2023

odooNextev commented Mar 22, 2023

primes2h commented Mar 24, 2023

[12.0][ADD] l10n_it_sale_commission_riba #3204

[12.0][ADD] l10n_it_sale_commission_riba #3204

Conversation

odooNextev commented Mar 7, 2023

elvise commented Mar 7, 2023

odooNextev commented Mar 7, 2023

elvise commented Mar 7, 2023

stenext commented Mar 7, 2023

elvise commented Mar 7, 2023

odooNextev commented Mar 7, 2023

elvise commented Mar 7, 2023

francesco-ooops commented Mar 7, 2023

stenext commented Mar 7, 2023

francesco-ooops commented Mar 7, 2023

stenext commented Mar 7, 2023

francesco-ooops commented Mar 7, 2023

stenext commented Mar 7, 2023

tafaRU commented Mar 7, 2023

francesco-ooops commented Mar 7, 2023

stenext commented Mar 7, 2023

stenext commented Mar 7, 2023

primes2h commented Mar 7, 2023

stenext commented Mar 7, 2023

primes2h commented Mar 7, 2023

stefano-ooops left a comment

Choose a reason for hiding this comment

francesco-ooops commented Mar 9, 2023

This comment was marked as resolved.

matteonext left a comment

Choose a reason for hiding this comment

francesco-ooops commented Mar 9, 2023

francesco-ooops commented Mar 13, 2023

stenext commented Mar 13, 2023 • edited Loading

francesco-ooops commented Mar 13, 2023

odooNextev commented Mar 22, 2023

primes2h commented Mar 24, 2023

stenext commented Mar 13, 2023 •

edited

Loading