[16.0][MIG]l10n_it_riba_sale_commission: migration to v16 #4403
Conversation
|
/ocabot migration l10n_it_riba_sale_commission |
There was a problem hiding this comment.
Grazie della PR!
Ho verificato che migrando da 14.0 con OpenUpgrade i dati vengono preservati.
Ho notato che alcuni commit di 14.0 mancano, tipo ead038a, potresti includerli? Se possibile andrebbero messi nel punto giusto della storia, quindi prima dei commit di migrazione
Puoi schiacciare i commit dei bot come indicato in https://github.com/OCA/maintainer-tools/wiki/Merge-commits-in-pull-requests#mergesquash-the-commits-generated-by-bots-or-weblate?
|
@odooNextev, @aleuffre, @renda-dev, @PicchiSeba visto che mantenete il modulo in |
| "application": False, | ||
| "installable": True, | ||
| "depends": [ | ||
| "sale_commission", |
There was a problem hiding this comment.
Questa dipendenza è stata splittata in 15.0 (vedi OCA/commission#387) in diversi moduli, ora dovrebbe essere sufficiente account_commission.
There was a problem hiding this comment.
Grazie, ero partito da un branch non aggiornato e quindi non ero correttamente allineato alle ultime modifiche. Il modulo dipende da OCA/commission#561 che al momento è in attesa di reviews. Nei prossimi giorni provo a verificare quanto rimane da correggere su questa pr.
matteonext
force-pushed
the
16.0-mig-l10n_it_riba_sale_commission
branch
from
c5faa0f to
75f7b12
Compare
test-requirements.txt
Outdated
| @@ -0,0 +1 @@ | |||
| odoo16-addon-commission@git+https://github.com/OCA/commission/pull/561/head#subdirectory=setup/commission | |||
There was a problem hiding this comment.
I test non si stanno eseguendo per questo errore
fatal: repository 'OCA/commission#561' not found
(da https://github.com/OCA/l10n-italy/actions/runs/12182965960/job/33983246738?pr=4403#step:4:192)
probabile che sia perché questo è da correggere:
| odoo16-addon-commission@git+https://github.com/OCA/commission/pull/561/head#subdirectory=setup/commission | |
| odoo-addon-commission @ git+https://github.com/OCA/commission.git@refs/pull/561/head#subdirectory=setup/commission |
(vedi https://github.com/OCA/maintainer-tools/wiki/Use-temporary-reference(s)-to-another-pull-request(s))
Qualche test fallirà comunque per #4461, ma almeno il resto dei test verranno eseguiti.
|
/ocabot rebase |
Currently translated at 100.0% (8 of 8 strings) Translation: l10n-italy-14.0/l10n-italy-14.0-l10n_it_riba_sale_commission Translate-URL: https://translation.odoo-community.org/projects/l10n-italy-14-0/l10n-italy-14-0-l10n_it_riba_sale_commission/it/
Currently translated at 100.0% (8 of 8 strings) Translation: l10n-italy-14.0/l10n-italy-14.0-l10n_it_riba_sale_commission Translate-URL: https://translation.odoo-community.org/projects/l10n-italy-14-0/l10n-italy-14-0-l10n_it_riba_sale_commission/it/
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate. Translation: l10n-italy-14.0/l10n-italy-14.0-l10n_it_riba_sale_commission Translate-URL: https://translation.odoo-community.org/projects/l10n-italy-14-0/l10n-italy-14-0-l10n_it_riba_sale_commission/
Currently translated at 100.0% (9 of 9 strings) Translation: l10n-italy-14.0/l10n-italy-14.0-l10n_it_riba_sale_commission Translate-URL: https://translation.odoo-community.org/projects/l10n-italy-14-0/l10n-italy-14-0-l10n_it_riba_sale_commission/it/
Currently translated at 100.0% (9 of 9 strings) Translation: l10n-italy-14.0/l10n-italy-14.0-l10n_it_riba_sale_commission Translate-URL: https://translation.odoo-community.org/projects/l10n-italy-14-0/l10n-italy-14-0-l10n_it_riba_sale_commission/it/
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate. Translation: l10n-italy-14.0/l10n-italy-14.0-l10n_it_riba_sale_commission Translate-URL: https://translation.odoo-community.org/projects/l10n-italy-14-0/l10n-italy-14-0-l10n_it_riba_sale_commission/
Currently translated at 100.0% (9 of 9 strings) Translation: l10n-italy-14.0/l10n-italy-14.0-l10n_it_riba_sale_commission Translate-URL: https://translation.odoo-community.org/projects/l10n-italy-14-0/l10n-italy-14-0-l10n_it_riba_sale_commission/it/
…n the documentation
|
Congratulations, PR rebased to 16.0. |
OCA-git-bot
force-pushed
the
16.0-mig-l10n_it_riba_sale_commission
branch
from
083a7f0 to
7949bbe
Compare
| ] | ||
| if dates: | ||
| return date_payment_to < max(dates) | ||
| return super()._skip_future_payments(date_payment_to) |
There was a problem hiding this comment.
Non trovo questo metodo in super, puoi controllare?
Ho fatto qualche ricerca ed era stato introdotto in 14.0 da OCA/commission#441 ma in 16.0 non lo vedo, e non vedo nemmeno delle PR per portare OCA/commission#441 in 16.0.
There was a problem hiding this comment.
Grazie della review, si, effettivamente il metodo non è presente nella 16. Mi consigli di proporre una pr per portare la pr in account_commission oppure rimuovo il metodo?
@francesco-ooops cosa ne pensi?
There was a problem hiding this comment.
Grazie della review, si, effettivamente il metodo non è presente nella 16. Mi consigli di proporre una pr per portare la pr in account_commission oppure rimuovo il metodo?
Da quello che sto capendo di questi moduli questo override è un punto cardine del funzionamento di questo modulo, e se non esiste in super probabilmente vuole anche dire che non viene chiamato quando dovrebbe: secondo me sarebbe da portare OCA/commission#441 a 16.0.
Mi stupisce anche un po' che i test passino senza che questo metodo sia in super, quindi magari mi sto perdendo qualche pezzo.
There was a problem hiding this comment.
Ho proposto questa pr: OCA/commission#588
No description provided.