Skip to content

[11.0] mig l10n_it_fiscal_payment_term#634

Merged
eLBati merged 4 commits into
OCA:11.0from
linkiteurope:11.0-mig-l10n_it_fiscal_payment_term
Nov 8, 2018
Merged

[11.0] mig l10n_it_fiscal_payment_term#634
eLBati merged 4 commits into
OCA:11.0from
linkiteurope:11.0-mig-l10n_it_fiscal_payment_term

Conversation

@labaggio

Copy link
Copy Markdown

No description provided.

sergiocorato and others added 3 commits October 31, 2018 17:09
[FIX] removed untranslated po and change fields name for more general purpose

[FIX] ref to fiscal payment in xml demo

[IMP] add migration script
and openerp XML tag
@labaggio labaggio changed the title [11.0] mig l10n it fiscal payment term [11.0] mig l10n_it_fiscal_payment_term Oct 31, 2018
Comment thread l10n_it_fiscal_payment_term/__init__.py Outdated
from . import models


def pre_init_hook(cr):

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.

Questo penso possa essere rimosso, visto che l10n_it_fatturapa su 11.0 non esiste

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

In che senso non esiste?

Questo penso possa essere rimosso, visto che l10n_it_fatturapa su 11.0 non esiste

@eLBati Nel senso che non è ancora stato fatto il porting?
C'è un po' di confusione probabilmente. l10n_it_fatturapa ha tra le sue dipendenze questo modulo, ma in questo hook si controlla che l10_it_fatturapa sia installato per poi far un update sul db.
Se questo update è necessario, è più corretto spostare questo update in l10_it_fatturapa.
Che faccio?

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.

@eLBati Nel senso che non è ancora stato fatto il porting?

Esatto

@labaggio questo script è stato fatto perchè il modulo l10n_it_fiscal_payment_term è stato creato dopo che l10n_it_fatturapa era stato rilasciato sulla 10. Prima di l10n_it_fiscal_payment_term, i fatturapa.payment_term erano all'interno del modulo l10n_it_fatturapa. Quindi, per spostarli senza rompere i DB v10 dove il modulo era installato, era necessario questo script.

Quindi su v11 lo script in questione non è più necessario.

Comment thread l10n_it_fiscal_payment_term/README.rst Outdated
@@ -0,0 +1,48 @@
.. image:: https://img.shields.io/badge/licence-LGPL--3-blue.svg

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.

'summary': 'Electronic invoices payment',
'author': 'Davide Corio, Agile Business Group, Innoviu, '
'Odoo Italia Network, Odoo Community Association (OCA)',
'website': 'https://odoo-community.org',

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.

@eLBati eLBati added this to the 11.0 milestone Oct 31, 2018
@eLBati

eLBati commented Nov 8, 2018

Copy link
Copy Markdown
Member

Essendo un modulo beta, procedo

@eLBati eLBati merged commit fc7df90 into OCA:11.0 Nov 8, 2018
@Byloth Byloth deleted the 11.0-mig-l10n_it_fiscal_payment_term branch November 20, 2018 16:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants