Skip to content

[11.0] mig l10n it withholding tax#523

Closed
alessandrocamilli wants to merge 63 commits into
OCA:11.0from
alessandrocamilli:11.0-mig-l10n_it_withholding_tax
Closed

[11.0] mig l10n it withholding tax#523
alessandrocamilli wants to merge 63 commits into
OCA:11.0from
alessandrocamilli:11.0-mig-l10n_it_withholding_tax

Conversation

@alessandrocamilli

Copy link
Copy Markdown
Contributor

No description provided.

@alessandrocamilli alessandrocamilli mentioned this pull request Jun 26, 2018
35 tasks
@pedrobaeza pedrobaeza added this to the 11.0 milestone Jun 26, 2018
@alessandrocamilli alessandrocamilli force-pushed the 11.0-mig-l10n_it_withholding_tax branch from ef7d65a to 0e50d6f Compare June 26, 2018 17:27
@eLBati

eLBati commented Jul 6, 2018

Copy link
Copy Markdown
Member

@alessandrocamilli questo commit acc1559 è da includere?

@alessandrocamilli

Copy link
Copy Markdown
Contributor Author

@eLBati si è da includere

@alessandrocamilli alessandrocamilli force-pushed the 11.0-mig-l10n_it_withholding_tax branch 2 times, most recently from df5d81d to a790915 Compare July 25, 2018 17:31
@alessandrocamilli

Copy link
Copy Markdown
Contributor Author

@eLBati possiamo mergiare?

@eLBati

eLBati commented Aug 1, 2018

Copy link
Copy Markdown
Member

@alessandrocamilli quando terminato il processo di peer review

@eLBati eLBati left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Per me OK.
Ottimo per i test.
Fammi sapere se riesci a fare squash dei vecchi commit con lo stesso autore

@eLBati

eLBati commented Aug 17, 2018

Copy link
Copy Markdown
Member

@alessandrocamilli ci sarebbe da fare review di #523 ed eventualmente includere le modifiche in questa PR

@eLBati

eLBati commented Aug 22, 2018

Copy link
Copy Markdown
Member

Probabilmente intendevo #565 ...

@eLBati

eLBati commented Aug 27, 2018

Copy link
Copy Markdown
Member

Anche #578

@SimoRubi SimoRubi left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Puoi condensare i cambiamenti per la migrazione in un unico commit?

@tafaRU tafaRU added the needs fixing Has conflicts or is failing mandatory CI checks label Aug 28, 2018
@alessandrocamilli alessandrocamilli force-pushed the 11.0-mig-l10n_it_withholding_tax branch from 29bc696 to 3d83648 Compare August 29, 2018 09:52
@alessandrocamilli

Copy link
Copy Markdown
Contributor Author

@eLBati pr integrate
@SimoRubi i 3 commit ora sono stati uniti

@eLBati

eLBati commented Aug 29, 2018

Copy link
Copy Markdown
Member

Il commit di porting qual è?

@alessandrocamilli

Copy link
Copy Markdown
Contributor Author

@eLBati 3d83648

@eLBati

eLBati commented Sep 10, 2018

Copy link
Copy Markdown
Member

@alessandrocamilli ok, riesci a modificare il messaggio di commit in modo che sia chiaro che lo sia?

@alessandrocamilli alessandrocamilli force-pushed the 11.0-mig-l10n_it_withholding_tax branch from 3d83648 to e28fcac Compare September 12, 2018 05:14
@alessandrocamilli alessandrocamilli removed the needs fixing Has conflicts or is failing mandatory CI checks label Sep 12, 2018
Marco Calcagni and others added 27 commits September 24, 2018 19:08
* [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
Without the possibility to avoid bank journal, Withholding Tax Statement would appear in
Accounting Dashboard / Bank Reconciliation
under 'Minus Unreconciled Statement Lines'
…n order to get Travis green after weblate migration (OCA#519)

The problem was introduced by a manual manipulation of the .po files at OCA@949a109#diff-b06d4f56026a0366ca7fdd91f96ddf27
[FIX] 10.0 api compat

First beta

Travis check

Ref name

[FIX] Compute WT only if the invoice has wt lines

Added feature to unreconcile the wt move created by payment, and after it possible to re-reconcile
[FIX] add display_name on class  withholding.tax.move withholding.tax.statement

[FIX] add display_name on class  withholding.tax.move withholding.tax.statement

[FIX] removed select on class l10n_it_withholding_tax l10n_it_withholding_tax_payment

[FIX] add README
Disabled delete from tree view of wt statements and wt moves

Rates required

Disabled delete from tree view of wt statements and wt moves

Fix conflicts and changed code with new standards
Gli oggetti withholding.tax.move, withholding.tax.statement e withholding.tax.move.payment si possono eliminare dalla vista elendo dei relativi menu
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
Added tests

added PR OCA#565

integration OCA#578
@alessandrocamilli alessandrocamilli force-pushed the 11.0-mig-l10n_it_withholding_tax branch from e28fcac to 18e87e8 Compare September 24, 2018 17:08
@eLBati

eLBati commented Oct 3, 2018

Copy link
Copy Markdown
Member

Ci spostiamo su #601

@eLBati eLBati closed this Oct 3, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

9 participants