Merge lp://qastaging/~bruno-bottacini/openobject-italia/7-l10n_it_ricevute_bancarie into lp://qastaging/~openobject-italia-core-devs/openobject-italia/italian-addons-7.0

Proposed by bruno bottacini
Status: Needs review
Proposed branch: lp://qastaging/~bruno-bottacini/openobject-italia/7-l10n_it_ricevute_bancarie
Merge into: lp://qastaging/~openobject-italia-core-devs/openobject-italia/italian-addons-7.0
Diff against target: 3314 lines (+3179/-0)
26 files modified
l10n_it_ricevute_bancarie/AUTHORS.txt (+2/-0)
l10n_it_ricevute_bancarie/__init__.py (+32/-0)
l10n_it_ricevute_bancarie/__openerp__.py (+71/-0)
l10n_it_ricevute_bancarie/account/__init__.py (+24/-0)
l10n_it_ricevute_bancarie/account/account.py (+121/-0)
l10n_it_ricevute_bancarie/account/account_view.xml (+145/-0)
l10n_it_ricevute_bancarie/configurazione.py (+85/-0)
l10n_it_ricevute_bancarie/configurazione_view.xml (+50/-0)
l10n_it_ricevute_bancarie/i18n/it.po (+930/-0)
l10n_it_ricevute_bancarie/partner/__init__.py (+24/-0)
l10n_it_ricevute_bancarie/partner/partner.py (+37/-0)
l10n_it_ricevute_bancarie/partner/partner_view.xml (+23/-0)
l10n_it_ricevute_bancarie/riba.py (+401/-0)
l10n_it_ricevute_bancarie/riba_sequence.xml (+17/-0)
l10n_it_ricevute_bancarie/riba_view.xml (+180/-0)
l10n_it_ricevute_bancarie/riba_workflow.xml (+112/-0)
l10n_it_ricevute_bancarie/security/ir.model.access.csv (+15/-0)
l10n_it_ricevute_bancarie/wizard/__init__.py (+30/-0)
l10n_it_ricevute_bancarie/wizard/riba_file_export.py (+260/-0)
l10n_it_ricevute_bancarie/wizard/riba_file_export.xml (+32/-0)
l10n_it_ricevute_bancarie/wizard/wizard_accreditation.py (+137/-0)
l10n_it_ricevute_bancarie/wizard/wizard_accreditation.xml (+41/-0)
l10n_it_ricevute_bancarie/wizard/wizard_emissione_riba.py (+131/-0)
l10n_it_ricevute_bancarie/wizard/wizard_emissione_riba.xml (+50/-0)
l10n_it_ricevute_bancarie/wizard/wizard_unsolved.py (+184/-0)
l10n_it_ricevute_bancarie/wizard/wizard_unsolved.xml (+45/-0)
To merge this branch: bzr merge lp://qastaging/~bruno-bottacini/openobject-italia/7-l10n_it_ricevute_bancarie
Reviewer Review Type Date Requested Status
Lorenzo Battistini Disapprove
Review via email: mp+171085@code.qastaging.launchpad.net
To post a comment you must log in.
Revision history for this message
Lorenzo Battistini (elbati) wrote :

Ciao Bruno,

puoi riassumere le modifiche principali che hai fatto rispetto alla 6.1?

Ad esempio vedo che hai rimosso la dipendenza da 'account_due_list', ma nel 'fields_view_get' alla riga 269 del diff ci sono ancora dei riferimenti.

Alla riga 3257 mi sembra ci sia la modifica che per 6.1 è nel seguente MP
https://code.launchpad.net/~bruno-bottacini/openobject-italia/6.1-fix-bug-1167773/+merge/158863
Hai un'opinione riguardo al commento che ti avevo lasciato?

Grazie

review: Needs Information
219. By bruno bottacini

rimossa la dipendenza da account_due_list

Revision history for this message
bruno bottacini (bruno-bottacini) wrote :

Ciao Lorenzo,

è passato un po' di tempo, ma in sostanza ho fatto un mero porting della versione 6.1 con l'aggiunta, per poter rimuovere la dipendeza da account_due_list, dei campi utilizzati ma non più presenti perchè definiti in quel modulo appunto.

ora ho rimosso la dipendenza da account_due_list (un mio errore per la fretta).
invece mi era scappato il commento sulla versione 6.1, analizzo il problema.

grazie

Revision history for this message
Marzi Gianpaolo (g-marzi) wrote :

Sono fuori dall'ufficio da Gio 18/07/2013 fino a Dom 21/07/2013.

Nota: Questa è una risposta automatizzata al messaggio "Re:
[Openobject-italia-core-devs] [Merge]
lp:~bruno-bottacini/openobject-italia/7-l10n_it_ricevute_bancarie into
lp:openobject-italia/7.0" inviata il 19/07/2013 10.23.30.

Questa è l'unica notifica che verrà ricevuta mentre la persona è assente.

Revision history for this message
Lorenzo Battistini (elbati) wrote :

A proposito di 'account_due_list':
https://code.launchpad.net/~agilebg/account-payment/adding_account_due_list_7
sulla 7 esisterà

Quindi possiamo evitare di dover modificare 'l10n_it_ricevute_bancarie' per rimuovere la dipendenza

Revision history for this message
Davide Corio (enlightx-deactivatedaccount) wrote :

ho testato il branch, ma non mi è chiaro come possa funzionare, siccome dipende da strutture non più presenti su v7, come ad esempio res.partner.address.

Partendo dal branch di Bruno, ho fatto questo:

https://code.launchpad.net/~enlightx/openobject-italia/7.0-l10n_it_ricevute_bancarie

per il quale avevo erroneamente creato un MP verso openobject-italia/7.0.

Revision history for this message
Davide Corio (enlightx-deactivatedaccount) wrote :

Se questo branch ha ancora senso posso creare un MP verso questo e continuiamo a lavorare qui.

Revision history for this message
Lorenzo Battistini (elbati) wrote :

Bruno, se ci sei batti un colpo :-)

Altrimenti spostiamo tutto su un altro branch posseduto da 'openobject-italia-core-devs' o da qualcuno di attivo.

Revision history for this message
bruno bottacini (bruno-bottacini) wrote :

scusatemi, per me potete pure spostarlo o stare qui

Revision history for this message
Lorenzo Battistini (elbati) wrote :

Suggerirei di partire comunque da un altro branch, inserendo unicamente le modifiche per fare andare il modulo sulla 7. Eventuali bachi o migliorie li gestirei in proposte a parte.

review: Disapprove
Revision history for this message
Davide Corio (enlightx-deactivatedaccount) wrote :

riproporrei quindi le modifiche che ho fatto, ma partendo dalla versione ufficiale per 6.1

Revision history for this message
Lorenzo Battistini (elbati) wrote :

On 10/22/2013 12:25 PM, Davide Corio wrote:
> riproporrei quindi le modifiche che ho fatto, ma partendo dalla versione ufficiale per 6.1

Perfetto, grazie.

Revision history for this message
Lorenzo Battistini (elbati) wrote :

This project is now hosted on https://github.com/OCA/l10n-italy. Please move your proposal there. This guide may help you https://github.com/OCA/maintainers-tools/wiki/How-to-move-a-Merge-Proposal-to-GitHub

Unmerged revisions

219. By bruno bottacini

rimossa la dipendenza da account_due_list

218. By bruno bottacini

add l10n_it_ricevute_bancarie

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches