search

OCA / l10n-italy

Odoo Italian localization
https://www.odoo-italia.org
GNU Affero General Public License v3.0
143 stars 298 forks source link

[16.0][FIX] customer bank account for riba payment from commercial entity #4079

Open alessandro-fiorino opened 3 months ago

alessandro-fiorino commented 3 months ago

I conti correnti sono collegati al contatto dell'entità commerciale principale, per cui il dominio dovrebbe essere 'partner_id' = 'commercial_partner_id' non 'partner_id'

alessandro-fiorino commented 3 months ago

Gli errori sono relativi a FAIL: TestCurrencyRateUpdate.test_foreign_base_currency ?

SirAionTech commented 3 months ago

Grazie della PR! Puoi aprire una issue (se non ce n'è già una) descrivendo i passi per riprodurre l'errore? Vedi https://github.com/OCA/l10n-italy/wiki/Team-di-sviluppo#apertura-issue.

Gli errori sono relativi a FAIL: TestCurrencyRateUpdate.test_foreign_base_currency ?

Potrebbe essere di nuovo https://github.com/OCA/l10n-italy/issues/3801, @Borruso puoi verificare?

@alessandro-fiorino potrebbe essere un errore temporaneo, puoi provare a far ripartire la CI? Puoi farlo cambiando l'hash dell'ultimo commit (non sono necessarie modifiche):

git commit --amend
git push --force
alessandro-fiorino commented 3 months ago

Aggiunta issue che questo fix dovrebbe sistemare https://github.com/OCA/l10n-italy/issues/4080

alessandro-fiorino commented 3 months ago

@alessandro-fiorino potrebbe essere un errore temporaneo, puoi provare a far ripartire la CI? Puoi farlo cambiando l'hash dell'ultimo commit (non sono necessarie modifiche):

git commit --amend
git push --force

Fatto ma sempre errore sul tasso di cambio.

francesco-ooops commented 3 months ago

@alessandro-fiorino penso che se domani fai ripartire i test sarà corretto, dato che è un errore temporaneo del modulo

alessandro-fiorino commented 3 months ago

Ora il test OCB è ok, il test Odoo dà problemi con TestBillOfEntry

alessandro-fiorino commented 2 months ago

I test sono tutti ok ora!

alessandro-fiorino commented 2 months ago

Qualche possibilità di merge prossimamente ?

SirAionTech commented 2 months ago

Qualche possibilità di merge prossimamente ?

Per fare il merge servono delle revisioni. Un buon modo per avere revisioni è farne a PR di altri contributor, oppure chiederle a chi ha già lavorato nel modulo che stai modificando.

Per semplificare il lavoro dei revisori, è necessario sapere come si può riprodurre il problema e qual è il comportamento atteso/attuale; puoi aggiungere queste info a https://github.com/OCA/l10n-italy/issues/4080? In pratica è la struttura che Github ti ha proposto quando hai aperto la issue.