Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[14.0][ADD] Aggiunto modulo l10n_it_ricevute_bancarie_enterprise #2139

Conversation

Borruso
Copy link
Contributor

@Borruso Borruso commented Feb 26, 2021

Porting del modulo l10n_it_ricevute_bancarie_for_enterprise dalla versione 12.0 alla 14.0
Poiché il modulo ha come dipendenza l10n_it_ricevute_bancarie per provarlo alla 14 bisogna installare la PR 2138

--
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

TheMule71 added a commit to odoo-italia/l10n-italy that referenced this pull request Feb 26, 2021
@TheMule71 TheMule71 mentioned this pull request Feb 26, 2021
76 tasks
"license": "AGPL-3",
"depends": ["account_accountant", "l10n_it_ricevute_bancarie"],
"data": ["views/riba_view.xml"],
"installable": True,
Copy link
Member

Choose a reason for hiding this comment

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

maintainers?

@eLBati
Copy link
Member

eLBati commented Feb 27, 2021

Porting del modulo l10n_it_ricevute_bancarie_for_enterprise dalla versione 12.0

Sulla 12 non mi pare ci sia.

Come nome, ci sono altri casi simili in OCA per eventualmente uniformarci?

@sergiocorato sergiocorato changed the title [ADD] Aggiunto modulo l10n_it_ricevute_bancarie_for_enterprise [14.0][ADD] Aggiunto modulo l10n_it_ricevute_bancarie_for_enterprise Mar 4, 2021
@dcorio
Copy link
Contributor

dcorio commented Apr 30, 2021

Propongo di eliminare quel "for" dal nome, così da uniformarci ai moduli enterprise

@Borruso Borruso force-pushed the 14.0-mig-l10n_it_ricevute_bancarie_for_enterprise branch from 4b27d2f to cf62de2 Compare April 30, 2021 10:30
@Borruso Borruso changed the title [14.0][ADD] Aggiunto modulo l10n_it_ricevute_bancarie_for_enterprise [14.0][ADD] Aggiunto modulo l10n_it_ricevute_bancarie_enterprise May 4, 2021
@Borruso Borruso force-pushed the 14.0-mig-l10n_it_ricevute_bancarie_for_enterprise branch from cf62de2 to e6ec1a7 Compare June 4, 2021 13:44
TheMule71 added a commit to odoo-italia/l10n-italy that referenced this pull request Jun 6, 2021
TheMule71 added a commit to odoo-italia/l10n-italy that referenced this pull request Jun 18, 2021
TheMule71 added a commit to odoo-italia/l10n-italy that referenced this pull request Jun 18, 2021
TheMule71 added a commit to odoo-italia/l10n-italy that referenced this pull request Jul 2, 2021
TheMule71 added a commit to odoo-italia/l10n-italy that referenced this pull request Jul 2, 2021
TheMule71 added a commit to odoo-italia/l10n-italy that referenced this pull request Jul 9, 2021
TheMule71 added a commit to odoo-italia/l10n-italy that referenced this pull request Jul 9, 2021
TheMule71 added a commit to odoo-italia/l10n-italy that referenced this pull request Jul 9, 2021
TheMule71 added a commit to odoo-italia/l10n-italy that referenced this pull request Jul 9, 2021
TheMule71 added a commit to odoo-italia/l10n-italy that referenced this pull request Jul 17, 2021
TheMule71 added a commit to odoo-italia/l10n-italy that referenced this pull request Jul 20, 2021
TheMule71 added a commit to odoo-italia/l10n-italy that referenced this pull request Jul 30, 2021
TheMule71 added a commit to odoo-italia/l10n-italy that referenced this pull request Jul 30, 2021
TheMule71 added a commit to odoo-italia/l10n-italy that referenced this pull request Aug 6, 2021
TheMule71 added a commit to odoo-italia/l10n-italy that referenced this pull request Aug 20, 2021
TheMule71 added a commit to odoo-italia/l10n-italy that referenced this pull request Sep 3, 2021
TheMule71 added a commit to odoo-italia/l10n-italy that referenced this pull request Sep 17, 2021
TheMule71 added a commit to odoo-italia/l10n-italy that referenced this pull request Oct 1, 2021
TheMule71 added a commit to odoo-italia/l10n-italy that referenced this pull request Oct 8, 2021
TheMule71 added a commit to odoo-italia/l10n-italy that referenced this pull request Oct 15, 2021
@Borruso Borruso force-pushed the 14.0-mig-l10n_it_ricevute_bancarie_for_enterprise branch from e6ec1a7 to 3ed272e Compare November 19, 2021 13:29
@Borruso Borruso requested review from eLBati and OpenCode November 19, 2021 13:29
Copy link
Contributor

@scigghia scigghia left a comment

Choose a reason for hiding this comment

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

👍

Copy link
Member

@SimoRubi SimoRubi left a comment

Choose a reason for hiding this comment

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

Riporto quanto detto stamattina in chiamata.
Questa PR non è mergiabile perché dipende dal modulo account_accountant che è enterprise, però il problema descritto va in qualche modo risolto.

In generale bisogna fare in modo che il modulo account_accountant sposti anche questo menu.
Visto che il modulo sposta i menu in modo statico semplicemente cambiando il parent dei menu che conosce (https://github.com/odoo/enterprise/blob/693ea0e9f35738a09465f0b628f6678ca8f4e601/account_accountant/data/account_accountant_data.xml#L10-L28), secondo me l'unica soluzione è mettere il menu delle RiBa sotto uno dei menù esistenti e verrà così spostato automaticamente.

Può comunque valere la pena indagare se ci sono altre soluzioni.

TheMule71 added a commit to TheMule71/l10n-italy that referenced this pull request Jan 21, 2022
TheMule71 added a commit to TheMule71/l10n-italy that referenced this pull request Jan 21, 2022
@Borruso Borruso closed this Feb 4, 2022
TheMule71 added a commit to odoo-italia/l10n-italy that referenced this pull request Jun 29, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants