[14.0][ADD] l10n_it_riba_sale_commission #3230

odooNextev · 2023-03-23T16:20:17Z

No description provided.

odooNextev · 2023-03-24T07:43:59Z

@tafaRU sapresti dirci come poter risolvere questo problema?
https://github.com/OCA/l10n-italy/actions/runs/4502926903/jobs/7925438831?pr=3230#step:6:5
Fondamentalmente il problema è che il nostro modulo dipende da l10n_it_ricevute_bancarie che è in stato alpha, però non dovrebbe aver raggiunto uno stato superiore visto che è stato pubblicato da un po'?
Perdonami se sto sbagliando qualcosa, ma non conosco bene gli stati e la loro assegnazione

tafaRU · 2023-03-24T08:29:28Z

@tafaRU sapresti dirci come poter risolvere questo problema?

Abbiamo tre strade che elenco in ordine di mia preferenza:

promuovere l10n_it_ricevute_bancarie a Beta
impostare esplicitamente in l10n_it_sale_commission_riba la chiave development_status a Alpha
disabilitare il controllo a livello di repository sfruttando https://github.com/OCA/oca-addons-repo-template/blob/3a63f6cc9d6825b4d5ce74c85754659de3fcca0b/copier.yml#L134

Per portare avanti il punto 1 devi creare una PR in cui semplicemente cambi il valore della la chiave development_status. Per ulteriori info al riguardo puoi fare riferimento a https://github.com/OCA/odoo-community.org/blob/master/website/Contribution/oca_module_lifecycle_development_status.rst

odooNextev · 2023-03-24T13:17:17Z

@primes2h abbiamo modificato il nome del modulo come avevi richiesto per la v14.0, tra poco faccio anche la v12.0
Abbiamo aggiunto il dev status ad alpha e chiamato la PR con ADD invece che MIG perchè effettivamente il modulo della 12.0 non è ancora stato mergiato

GSLabIt · 2023-03-30T09:08:40Z

l10n_it_riba_sale_commission/i18n/it.po

+msgstr "Liquidazione creata"
+
+#. module: l10n_it_riba_sale_commission 
+#: model:ir.model.fields,field_description:l10n_it_riba_sale_commission .field_account_invoice__no_commission


c'è uno spazio di troppo, verifica anche il resto del file perché si ripete.

stefano-ooops

testato ok

l10n_it_riba_sale_commission/README.rst

GSLabIt · 2023-04-04T14:01:59Z

l10n_it_riba_sale_commission/readme/DESCRIPTION.rst

+provvigioni agente solamente trascorsi i giorni configurati nel campo
+"Giorni di sicurezza" nella configurazione ri.ba.
+
+E' possibile inserire aggiungere "Senza commissioni" ad una fattura, in modo


@odooNextev scusate la pedanteria, se potete rimuovere uno tra "inserire" e "aggiungere" e sostituire "commissioni" con "provvigioni"

l10n_it_riba_sale_commission/views/invoice_no_commission.xml

l10n_it_riba_sale_commission/wizard/wizard_riba_commissions_settle.py

francesco-ooops

dopo queste dovremmo essere a posto

francesco-ooops · 2023-04-11T12:05:45Z

l10n_it_riba_sale_commission/readme/DESCRIPTION.rst

+provvigioni agente solamente trascorsi i giorni configurati nel campo
+"Giorni di sicurezza" nella configurazione ri.ba.
+
+E' possibile inserire aggiungere "Senza commissioni" ad una fattura, in modo


@odooNextev scusate la pedanteria, se potete rimuovere uno tra "inserire" e "aggiungere" e sostituire "commissioni" con "provvigioni"

francesco-ooops · 2023-04-11T12:06:31Z

l10n_it_riba_sale_commission/readme/DESCRIPTION.rst

@@ -0,0 +1,23 @@
+**Italiano**
+
+Il modulo permette di integrare il pagamento riba con le commissioni agenti.


sostituire "commissioni" con "provvigioni"

francesco-ooops · 2023-04-14T08:14:41Z

@OCA/local-italy-maintainers possiamo mergiare?

primes2h · 2023-04-14T08:18:48Z

@francesco-ooops in giornata vedo di dargli un'occhiata.

primes2h

Qualche suggerimento e correzione della parte documentale secondo le linee guida.

l10n_it_riba_sale_commission/__manifest__.py

primes2h · 2023-04-17T10:04:37Z

l10n_it_riba_sale_commission/__manifest__.py

+    "summary": "Modulo ponte tra provvigioni agenti e Ri.Ba",
+    "version": "14.0.1.0.0",
+    "development_status": "Alpha",
+    "category": "Sales",


Suggested change

"category": "Sales",

"category": "Localization/Italy",

l10n_it_riba_sale_commission/readme/DESCRIPTION.rst

primes2h · 2023-04-17T14:59:54Z

l10n_it_riba_sale_commission/wizard/wizard_riba_commissions_settle.py

+        # rimuove le righe delle fatture che hanno impostato flag "no_commission"
+        # e che abbiano Ri.Ba come metodo di pagamento da quelle recuperate
+        # con il metodo originale


Il commento andrebbe in inglese, come quello appena sopra.

primes2h · 2023-04-17T15:00:10Z

l10n_it_riba_sale_commission/wizard/wizard_riba_commissions_settle.py

+            # rimuove le righe se ri.ba è insoluta o nel caso sia sbf non siano
+            # passati almeno i giorni di sicurezza da data di scadenza del pagamento
+            # per tenersi un margine e verificare che sia stata pagata


Vedi sopra.

l10n_it_riba_sale_commission/models/account_move.py

l10n_it_riba_sale_commission/i18n/it.po

francesco-ooops · 2023-04-21T07:49:28Z

@odooNextev riuscite a fixare?

l10n_it_riba_sale_commission/i18n/it.po

francesco-ooops · 2023-04-26T08:18:03Z

@primes2h tutto ok ora?

primes2h · 2023-04-26T15:13:54Z

l10n_it_riba_sale_commission/wizard/wizard_riba_commissions_settle.py

+            # removes lines if Riba is unsolved or in case it is sbf and at least the
+            # security days have not passed since the payment due date to to keep a
+            # days margin and verify that it has been paid


Attenzione che unsolved è un falso amico.
Inoltre sarebbe sempre meglio usare lo stesso termine utilizzato nel resto del codice ("safety").
P.S.: c'era un doppio "to".

Suggested change

# removes lines if Riba is unsolved or in case it is sbf and at least the

# security days have not passed since the payment due date to to keep a

# days margin and verify that it has been paid

# removes lines if RiBa is past due or in case it is subject to collection

# and at least the safety days have not passed since the payment due date,

# to keep a margin and verify that it has been paid.

odooNextev · 2023-04-27T08:24:10Z

@primes2h dovremmo aver sistemato. Ora è ok?

primes2h · 2023-04-27T13:06:19Z

@odooNextev per richiedere di nuovo la revisione della PR dopo una modifica, invece di un messaggio ti conviene fare così:

Dopo il clic sul simbolo, cerca la persona alla quale vuoi richiedere la revisione (o aggiornare una esistente).

P.S.: visto il commento che aveva fatto qui #3204 (comment) e le modifiche conseguenti, ti conviene chiedere una nuova revisione anche a @tafaRU.

tafaRU · 2023-04-28T08:07:53Z

/ocabot merge nobump

OCA-git-bot · 2023-04-28T08:13:00Z

What a great day to merge this nice PR. Let's do it!
Prepared branch 14.0-ocabot-merge-pr-3230-by-tafaRU-bump-nobump, awaiting test results.

SirTakobi · 2023-04-28T08:17:32Z

@odooNextev per richiedere di nuovo la revisione della PR dopo una modifica, invece di un messaggio ti conviene fare così:

Dopo il clic sul simbolo, cerca la persona alla quale vuoi richiedere la revisione (o aggiornare una esistente).

I normali contributor non vedono il ⚙️ ma possono solo richiedere la review di chi è già tra i reviewer:

OCA-git-bot · 2023-04-28T08:39:54Z

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

primes2h · 2023-05-03T09:03:50Z

@odooNextev per richiedere di nuovo la revisione della PR dopo una modifica, invece di un messaggio ti conviene fare così:

Dopo il clic sul simbolo, cerca la persona alla quale vuoi richiedere la revisione (o aggiornare una esistente).

I normali contributor non vedono il gear ma possono solo richiedere la review di chi è già tra i reviewer:

👍
Hai ragione, grazie per l'osservazione.

odooNextev force-pushed the 14.0-mig-l10n_italy_sale_commission_riba branch from 201101a to 09e7165 Compare March 23, 2023 16:29

odooNextev changed the title ~~[14.0][mig]l10n italy sale commission riba~~ [14.0][mig]l10n_italy_sale_commission_riba Mar 24, 2023

This was referenced Mar 24, 2023

[14.0][IMP] l10n_it_ricevute_bancarie: change development_status to "Beta" #3234

Closed

[14.0][IMP] l10n_it_ricevute_bancarie: change development_status to "Beta" #3235

Merged

odooNextev force-pushed the 14.0-mig-l10n_italy_sale_commission_riba branch 2 times, most recently from ae8e429 to bdd632b Compare March 24, 2023 13:05

odooNextev changed the title ~~[14.0][mig]l10n_italy_sale_commission_riba~~ [14.0][MIG] l10n_it_riba_sale_commission Mar 24, 2023

odooNextev changed the title ~~[14.0][MIG] l10n_it_riba_sale_commission~~ [14.0][ADD] l10n_it_riba_sale_commission Mar 24, 2023

odooNextev force-pushed the 14.0-mig-l10n_italy_sale_commission_riba branch 3 times, most recently from a28bd40 to 641b248 Compare March 24, 2023 16:44

primes2h mentioned this pull request Mar 29, 2023

16.0 mig l10n it ricevute bancarie #3131

Closed

GSLabIt suggested changes Mar 30, 2023

View reviewed changes

odooNextev force-pushed the 14.0-mig-l10n_italy_sale_commission_riba branch from 641b248 to dce7755 Compare March 30, 2023 09:32

stefano-ooops approved these changes Apr 3, 2023

View reviewed changes

GSLabIt reviewed Apr 4, 2023

View reviewed changes

odooNextev force-pushed the 14.0-mig-l10n_italy_sale_commission_riba branch 2 times, most recently from 5374391 to 90f9af7 Compare April 5, 2023 12:52

francesco-ooops approved these changes Apr 11, 2023

View reviewed changes

odooNextev force-pushed the 14.0-mig-l10n_italy_sale_commission_riba branch from 90f9af7 to 1ce181d Compare April 11, 2023 12:24

GSLabIt approved these changes Apr 14, 2023

View reviewed changes

OCA-git-bot added the approved label Apr 14, 2023

primes2h requested changes Apr 17, 2023

View reviewed changes

OCA-git-bot removed the approved label Apr 17, 2023

odooNextev force-pushed the 14.0-mig-l10n_italy_sale_commission_riba branch from 8e0b989 to 426e8f5 Compare April 21, 2023 13:14

primes2h requested changes Apr 21, 2023

View reviewed changes

l10n_it_riba_sale_commission/i18n/it.po Outdated Show resolved Hide resolved

odooNextev force-pushed the 14.0-mig-l10n_italy_sale_commission_riba branch from 426e8f5 to 92298f3 Compare April 21, 2023 14:12

primes2h requested changes Apr 26, 2023

View reviewed changes

[14.0][ADD] l10n_it_riba_sale_commission

187bd0f

odooNextev force-pushed the 14.0-mig-l10n_italy_sale_commission_riba branch from 92298f3 to 187bd0f Compare April 27, 2023 06:50

primes2h approved these changes Apr 27, 2023

View reviewed changes

OCA-git-bot added the approved label Apr 27, 2023

tafaRU approved these changes Apr 28, 2023

View reviewed changes

OCA-git-bot added the bot is merging ⏳ label Apr 28, 2023

odooNextev requested review from francesco-ooops and stefano-ooops April 28, 2023 08:26

OCA-git-bot merged commit 6ee7873 into OCA:14.0 Apr 28, 2023

OCA-git-bot added merged 🎉 and removed bot is merging ⏳ labels Apr 28, 2023

francesco-ooops added migration new module and removed migration labels Jan 10, 2024

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[14.0][ADD] l10n_it_riba_sale_commission #3230

[14.0][ADD] l10n_it_riba_sale_commission #3230

odooNextev commented Mar 23, 2023

odooNextev commented Mar 24, 2023

tafaRU commented Mar 24, 2023

odooNextev commented Mar 24, 2023 •

edited

Loading

GSLabIt Mar 30, 2023

stefano-ooops left a comment

GSLabIt Apr 4, 2023

francesco-ooops Apr 11, 2023

francesco-ooops left a comment

francesco-ooops Apr 11, 2023

francesco-ooops Apr 11, 2023

francesco-ooops commented Apr 14, 2023

primes2h commented Apr 14, 2023

primes2h left a comment

primes2h Apr 17, 2023

primes2h Apr 17, 2023

primes2h Apr 17, 2023

francesco-ooops commented Apr 21, 2023

francesco-ooops commented Apr 26, 2023

primes2h Apr 26, 2023

odooNextev commented Apr 27, 2023

primes2h commented Apr 27, 2023 •

edited

Loading

tafaRU commented Apr 28, 2023

OCA-git-bot commented Apr 28, 2023

SirTakobi commented Apr 28, 2023

OCA-git-bot commented Apr 28, 2023

primes2h commented May 3, 2023

		@@ -0,0 +1,23 @@
		Italiano

		Il modulo permette di integrare il pagamento riba con le commissioni agenti.

[14.0][ADD] l10n_it_riba_sale_commission #3230

[14.0][ADD] l10n_it_riba_sale_commission #3230

Conversation

odooNextev commented Mar 23, 2023

odooNextev commented Mar 24, 2023

tafaRU commented Mar 24, 2023

odooNextev commented Mar 24, 2023 • edited Loading

Choose a reason for hiding this comment

stefano-ooops left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

francesco-ooops left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

francesco-ooops commented Apr 14, 2023

primes2h commented Apr 14, 2023

primes2h left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

francesco-ooops commented Apr 21, 2023

francesco-ooops commented Apr 26, 2023

Choose a reason for hiding this comment

odooNextev commented Apr 27, 2023

primes2h commented Apr 27, 2023 • edited Loading

tafaRU commented Apr 28, 2023

OCA-git-bot commented Apr 28, 2023

SirTakobi commented Apr 28, 2023

OCA-git-bot commented Apr 28, 2023

primes2h commented May 3, 2023

odooNextev commented Mar 24, 2023 •

edited

Loading

primes2h commented Apr 27, 2023 •

edited

Loading