[12.0] Migration: l10n_it_withholding_tax_payment #1281

Byloth · 2019-06-07T16:09:35Z

Porting del modulo l10n_it_withholding_tax_payment dalla versione 11.0 alla versione 12.0.

--
Confermo di aver firmato il CLA https://odoo-community.org/page/cla e di aver letto le linee guida su https://odoo-community.org/page/contributing

primes2h

Review (non di codice).

l10n_it_withholding_tax_payment/__manifest__.py

l10n_it_withholding_tax_payment/readme/DESCRIPTION.rst

l10n_it_withholding_tax_payment/README.rst

Byloth

Domanda per @alessandrocamilli... 😁

l10n_it_withholding_tax_payment/models/withholding_tax.py

l10n_it_withholding_tax_payment/__manifest__.py

warp10

LGTM

primes2h

Dopo queste piccole modifiche per me è 👍

l10n_it_withholding_tax_payment/models/withholding_tax.py

tafaRU · 2019-11-19T10:14:19Z

@Byloth hai interesse a mantenere separati gli ultimi tre commit o si possono accorpare?

Byloth · 2019-11-19T11:03:08Z

@Byloth hai interesse a mantenere separati gli ultimi tre commit o si possono accorpare?

Ad essere sincero, no... Mi è assolutamente indifferente!
Anzi... Li avrei già accorpati qualora fossero stati eseguiti dallo stesso utente.

So che, appunto, per prassi si usa accorpare commit consecutivi eseguiti da un medesimo utente...
In questo caso, però, non volevo togliere il commit fatto da @SilvioGregorini con il relativo riferimento e contributo.

Non nego che, fossi il lui, mi girerebbero notevolmente: un contributo È un contributo.

In ogni caso, è un mio (opinabile) punto di vista e non ho intenzione di aprire, a riguardo, un'intera discussione...

Per cui...

Esiste un modo per eseguire lo squash di più commit mantenendo i relativi riferimenti dei contributors?

... oppure...
Se per @SilvioGregorini non ci sono problemi, proseguo immediatamente con uno squash dei commit incriminati!

«Voi di dite quello che devo fare... E io lo faccio!» 😁

tafaRU · 2019-11-19T11:11:26Z

Esiste un modo per eseguire lo squash di più commit mantenendo i relativi riferimenti dei contributors?

Non che io sappia!

Al limite puoi accorpare il commit di migrazione con l'ultimo e lasciare quello di @SilvioGregorini.
Me vediamo prima cosa dice lui.

SilvioGregorini · 2019-11-19T11:16:32Z

Se per @SilvioGregorini non ci sono problemi, proseguo immediatamente con uno squash dei commit incriminati!

@Byloth figurati, per un refractoring di due stringhe in tutto!
"Squashi lei!"

Byloth · 2019-11-19T11:42:11Z

"Squashi lei!"

«Ah, congiuntivo! Aspetti!»

Ho risolto...
Ora sono felice! 🙃

l10n_it_withholding_tax_payment/models/withholding_tax.py

OpenCode · 2019-11-20T09:09:46Z

@tafaRU Hai da fare review oppure osservazioni tecniche? Altrimenti mergerei.

tafaRU · 2019-11-20T09:24:25Z

@Byloth non è bloccante (dal momento che Travis non lo segnala), ma sarebbe bello, se avessi voglia e tempo, anche in vista dei prossimi lavori, se uniformassi il tuo copyright alle line guida di OCA.
Grazie!

Byloth · 2019-11-20T10:07:14Z

@tafaRU intendi che dovrei rimuovere, solamente, (c)?

tafaRU · 2019-11-20T11:23:59Z

@tafaRU intendi che dovrei rimuovere, solamente, (c)?

sì, grazie @Byloth

Byloth · 2019-11-20T13:21:48Z

Faaatto! 😁

l10n_it_withholding_tax_payment/models/withholding_tax.py

eLBati · 2019-12-16T07:03:25Z

@Byloth Come mai tutti i file di traduzioni (insluso it.po) rimossi nel commit di migrazione?

Byloth · 2019-12-16T08:24:26Z

Le traduzioni non le facciamo gestire da Weblate?
Mi aspetto che, in autonomia, vada a ricreare tutti i file .po del caso, evitando eventuali conflitti... Sbaglio?

eLBati · 2019-12-16T10:04:02Z

@Byloth sì, però in questo modo rimouvi anche i termini eventualmente già tradotti

check wt_moves Added rules Show payment move after generation from wt moves More check for payment unreconciliation Company readonly in payment wt Company readonly in payment wt Fix conflict with translation

* [FIX] Travis check and readonly on company_id * [FIX] Travis check and readonly on company_id

odoo/tools/translate.py", line 539, in next raise Exception("malformed file: bad line: %s" % line) Exception: malformed file: bad line: # * l10n_it_withholding_tax Restore l10n_it_withholding_tax_payment/i18n/lt.po

Currently translated at 74.5% (35 of 47 strings) Translation: l10n-italy-11.0/l10n-italy-11.0-l10n_it_withholding_tax_payment Translate-URL: https://translation.odoo-community.org/projects/l10n-italy-11-0/l10n-italy-11-0-l10n_it_withholding_tax_payment/it/

Updated by Update PO files to match POT (msgmerge) hook in Weblate.

tafaRU

Thanks!

tafaRU · 2019-12-19T16:06:39Z

@primes2h attendiamo tua review prima di mergiare. Grazie!

l10n_it_withholding_tax_payment/models/withholding_tax.py

l10n_it_withholding_tax_payment/i18n/l10n_it_withholding_tax_payment.pot

primes2h

Giusto le ultime due che mi erano sfuggite e per me è 👍

l10n_it_withholding_tax_payment/i18n/l10n_it_withholding_tax_payment.pot

primes2h

Ottimo, grazie!

labaggio · 2019-12-20T10:37:32Z

Ottimo, grazie!

@primes2h faccio una proposta per le prossime volte.
Partendo dal presupposto che le revisioni sulle traduzioni sono assolutamente indispensabili, ottimizzerebbe un sacco i tempi se questo tipo di revisione fosse gestito da una PR (sulla PR) e non da tanti messaggi quante sono le stringhe da cambiare.
Per te o per chi sta facendo review significa cambiarlo direttamente nel codice così da non fare doppio lavoro, tu che scrivi il commento e il programmatore che deve riportare la modifica.
;)

labaggio · 2019-12-20T13:54:36Z

/ocabot merge

OCA-git-bot · 2019-12-20T13:54:58Z

Hey, thanks for contributing! Proceeding to merge this for you.
Prepared branch 12.0-ocabot-merge-pr-1281-by-labaggio-bump-no, awaiting test results.

Signed-off-by labaggio

OCA-git-bot · 2019-12-20T14:42:58Z

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

primes2h · 2019-12-20T15:09:35Z

Ottimo, grazie!

@primes2h faccio una proposta per le prossime volte.
Partendo dal presupposto che le revisioni sulle traduzioni sono assolutamente indispensabili, ottimizzerebbe un sacco i tempi se questo tipo di revisione fosse gestito da una PR (sulla PR) e non da tanti messaggi quante sono le stringhe da cambiare.
Per te o per chi sta facendo review significa cambiarlo direttamente nel codice così da non fare doppio lavoro, tu che scrivi il commento e il programmatore che deve riportare la modifica.
;)

In effetti quando le modifiche richieste iniziano a essere tante concordo con te che sarebbe meglio fare una PR sulla PR. 👍
Però confido anche che aumentando la consapevolezza sull'importanza degli aspetti documentali/funzionali/traduzione (e linee guida) diminuisca anche la necessità di doverle fare, è un lavoraccio. 😅

primes2h requested changes Jun 9, 2019

View reviewed changes

l10n_it_withholding_tax_payment/__manifest__.py Outdated Show resolved Hide resolved

l10n_it_withholding_tax_payment/readme/DESCRIPTION.rst Show resolved Hide resolved

primes2h mentioned this pull request Jun 9, 2019

[12.0] Migration: l10n_it_vat_registries_split_payment #1280

Merged

primes2h requested changes Jun 10, 2019

View reviewed changes

l10n_it_withholding_tax_payment/README.rst Outdated Show resolved Hide resolved

Byloth commented Jun 13, 2019

View reviewed changes

l10n_it_withholding_tax_payment/models/withholding_tax.py Outdated Show resolved Hide resolved

eLBati added the needs review label Jun 26, 2019

primes2h requested changes Aug 20, 2019

View reviewed changes

l10n_it_withholding_tax_payment/models/withholding_tax.py Outdated Show resolved Hide resolved

l10n_it_withholding_tax_payment/models/withholding_tax.py Outdated Show resolved Hide resolved

l10n_it_withholding_tax_payment/__manifest__.py Outdated Show resolved Hide resolved

OCA-git-bot mentioned this pull request Sep 30, 2019

Migration to version 12.0 #598

Closed

35 tasks

eLBati mentioned this pull request Oct 7, 2019

Manage WT for customer invoices #1483

Closed

2 tasks

warp10 approved these changes Nov 18, 2019

View reviewed changes

primes2h requested changes Nov 19, 2019

View reviewed changes

l10n_it_withholding_tax_payment/models/withholding_tax.py Outdated Show resolved Hide resolved

primes2h approved these changes Nov 19, 2019

View reviewed changes

OCA-git-bot added the approved label Nov 19, 2019

alessandrocamilli approved these changes Nov 19, 2019

View reviewed changes

OpenCode reviewed Nov 20, 2019

View reviewed changes

l10n_it_withholding_tax_payment/models/withholding_tax.py Outdated Show resolved Hide resolved

l10n_it_withholding_tax_payment/models/withholding_tax.py Outdated Show resolved Hide resolved

tafaRU requested changes Nov 20, 2019

View reviewed changes

l10n_it_withholding_tax_payment/models/withholding_tax.py Show resolved Hide resolved

OCA-git-bot removed the approved label Nov 20, 2019

Byloth requested a review from tafaRU December 5, 2019 14:58

alessandrocamilli and others added 10 commits December 19, 2019 15:02

Multicompany

0ed9e66

check wt_moves Added rules Show payment move after generation from wt moves More check for payment unreconciliation Company readonly in payment wt Company readonly in payment wt Fix conflict with translation

[FIX] Travis check and readonly on company_id (#25)

b6e70b7

* [FIX] Travis check and readonly on company_id * [FIX] Travis check and readonly on company_id

FIX l10n_it_withholding_tax

d3067f5

odoo/tools/translate.py", line 539, in next raise Exception("malformed file: bad line: %s" % line) Exception: malformed file: bad line: # * l10n_it_withholding_tax Restore l10n_it_withholding_tax_payment/i18n/lt.po

OCA Transbot updated translations from Transifex

0258373

[UPD] Update l10n_it_withholding_tax_payment.pot

0aacc78

porting l10n_it_withholding_tax to v11

80da0be

[UPD] Update l10n_it_withholding_tax_payment.pot

d3eac6a

Update translation files

259b979

Updated by Update PO files to match POT (msgmerge) hook in Weblate.

[ADD] icon.png

cdba4e0

Byloth requested a review from primes2h December 19, 2019 14:03

labaggio approved these changes Dec 19, 2019

View reviewed changes

tafaRU approved these changes Dec 19, 2019

View reviewed changes

primes2h reviewed Dec 19, 2019

View reviewed changes

Byloth requested a review from primes2h December 20, 2019 09:25

primes2h requested changes Dec 20, 2019

View reviewed changes

l10n_it_withholding_tax_payment/i18n/l10n_it_withholding_tax_payment.pot Outdated Show resolved Hide resolved

l10n_it_withholding_tax_payment/i18n/l10n_it_withholding_tax_payment.pot Outdated Show resolved Hide resolved

Byloth requested a review from primes2h December 20, 2019 10:09

primes2h approved these changes Dec 20, 2019

View reviewed changes

OCA-git-bot added the approved label Dec 20, 2019

[MIG] l10n_it_withholding_tax_payment: Migration to 12.0

f6f94e7

OCA-git-bot added a commit that referenced this pull request Dec 20, 2019

Merge PR #1281 into 12.0

b50b1ea

Signed-off-by labaggio

OCA-git-bot merged commit f6f94e7 into OCA:12.0 Dec 20, 2019

OCA-git-bot added the merged 🎉 label Dec 20, 2019

francesco-ooops added the no issue needed label Jan 24, 2024

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[12.0] Migration: l10n_it_withholding_tax_payment #1281

[12.0] Migration: l10n_it_withholding_tax_payment #1281

Byloth commented Jun 7, 2019

primes2h left a comment

Byloth left a comment

warp10 left a comment

primes2h left a comment

tafaRU commented Nov 19, 2019

Byloth commented Nov 19, 2019

tafaRU commented Nov 19, 2019

SilvioGregorini commented Nov 19, 2019

Byloth commented Nov 19, 2019

OpenCode commented Nov 20, 2019

tafaRU commented Nov 20, 2019

Byloth commented Nov 20, 2019

tafaRU commented Nov 20, 2019

Byloth commented Nov 20, 2019

eLBati commented Dec 16, 2019

Byloth commented Dec 16, 2019

eLBati commented Dec 16, 2019

tafaRU left a comment

tafaRU commented Dec 19, 2019

primes2h left a comment

primes2h left a comment

labaggio commented Dec 20, 2019

labaggio commented Dec 20, 2019

OCA-git-bot commented Dec 20, 2019

OCA-git-bot commented Dec 20, 2019

primes2h commented Dec 20, 2019

[12.0] Migration: l10n_it_withholding_tax_payment #1281

[12.0] Migration: l10n_it_withholding_tax_payment #1281

Conversation

Byloth commented Jun 7, 2019

primes2h left a comment

Choose a reason for hiding this comment

Byloth left a comment

Choose a reason for hiding this comment

warp10 left a comment

Choose a reason for hiding this comment

primes2h left a comment

Choose a reason for hiding this comment

tafaRU commented Nov 19, 2019

Byloth commented Nov 19, 2019

tafaRU commented Nov 19, 2019

SilvioGregorini commented Nov 19, 2019

Byloth commented Nov 19, 2019

OpenCode commented Nov 20, 2019

tafaRU commented Nov 20, 2019

Byloth commented Nov 20, 2019

tafaRU commented Nov 20, 2019

Byloth commented Nov 20, 2019

eLBati commented Dec 16, 2019

Byloth commented Dec 16, 2019

eLBati commented Dec 16, 2019

tafaRU left a comment

Choose a reason for hiding this comment

tafaRU commented Dec 19, 2019

primes2h left a comment

Choose a reason for hiding this comment

primes2h left a comment

Choose a reason for hiding this comment

labaggio commented Dec 20, 2019

labaggio commented Dec 20, 2019

OCA-git-bot commented Dec 20, 2019

OCA-git-bot commented Dec 20, 2019

primes2h commented Dec 20, 2019