OCA / l10n-italy

Odoo Italian localization
https://www.odoo-italia.org
GNU Affero General Public License v3.0
147 stars 301 forks source link

[14.0] l10n_it_account_balance_eu: delete get_xlsx_name #4048

Closed TennyMkt closed 5 months ago

TennyMkt commented 5 months ago

Viene tolta la funzione _get_xlsx_name e la sua chiamata in questo modulo. Di fatto veniva sovrascritta la stessa funzione presente in account_financial_report ma non serve perchè il nome del file xlsx è fisso.

OCA-git-bot commented 5 months ago

Hi @mktsrl, some modules you are maintaining are being modified, check this out!

TennyMkt commented 5 months ago

Non avevo notato però il messaggio del commit, puoi modificarlo per seguire https://github.com/OCA/odoo-community.org/blob/8afdb5a5ebfd3314e2fc836c9b9f644468ee044e/website/Contribution/CONTRIBUTING.rst#L1114?

Ho aggiornato il messaggio del commit

francesco-ooops commented 5 months ago

@mktsrl puoi procedere con il merge

mktsrl commented 5 months ago

@mktsrl puoi procedere con il merge

Non ho il tasto per fare il merge. Cosa manca?

francesco-ooops commented 5 months ago

@mktsrl come maintainer del modulo, potete usare i comandi di ocabot: https://github.com/OCA/oca-github-bot/blob/master/README.rst#id7

in questo caso direi /ocabot merge patch

OCA-git-bot commented 5 months ago

This PR looks fantastic, let's merge it! Prepared branch 14.0-ocabot-merge-pr-4048-by-francesco-ooops-bump-patch, awaiting test results.

OCA-git-bot commented 5 months ago

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

francesco-ooops commented 5 months ago

hehe volevo vedere se funzionava anche in un commento "non dedicato" :)

SirAionTech commented 5 months ago

hehe volevo vedere se funzionava anche in un commento "non dedicato" :)

Volevi vedere se il bot era attento :smile:, comunque se vuoi scrivere il comando senza eseguirlo puoi farlo con una citazione:

/ocabot merge patch

è così da https://github.com/OCA/oca-github-bot/pull/84.