search

OCA / l10n-italy

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

[14.0][FIX]assets_management: differentiate groups #4113

Closed PicchiSeba closed 2 months ago

PicchiSeba commented 2 months ago

Add an additional group that glues assets_management.group_asset_user and account.group_account_invoice together

matteoopenf commented 2 months ago

puoi aprire una issue per tracciare la cosa?

OCA-git-bot commented 2 months ago

This PR has the approved label and has been created more than 5 days ago. It should therefore be ready to merge by a maintainer (or a PSC member if the concerned addon has no declared maintainer). 🤖

francesco-ooops commented 2 months ago

@sergiocorato buona per il merge?

sergiocorato commented 2 months ago

/ocabot merge minor

OCA-git-bot commented 2 months ago

This PR looks fantastic, let's merge it! Prepared branch 14.0-ocabot-merge-pr-4113-by-sergiocorato-bump-minor, awaiting test results.

OCA-git-bot commented 2 months ago

@sergiocorato your merge command was aborted due to failed check(s), which you can inspect on this commit of 14.0-ocabot-merge-pr-4113-by-sergiocorato-bump-minor.

After fixing the problem, you can re-issue a merge command. Please refrain from merging manually as it will most probably make the target branch red.

OCA-git-bot commented 2 months ago

@sergiocorato your merge command was aborted due to failed check(s), which you can inspect on this commit of 14.0-ocabot-merge-pr-4113-by-sergiocorato-bump-minor.

After fixing the problem, you can re-issue a merge command. Please refrain from merging manually as it will most probably make the target branch red.

francesco-ooops commented 2 months ago

@SirAionTech dato che hai seguito la questione codecov, basta un rebase qui o c'è altro da fare? grazie!

SirAionTech commented 2 months ago

@SirAionTech dato che hai seguito la questione codecov, basta un rebase qui o c'è altro da fare? grazie!

Basta un rebase perché le modifiche sono già in 14.0 (https://github.com/OCA/l10n-italy/commit/33646874d5de92f8e69be6ac2474c1e798d6746c).

PicchiSeba commented 2 months ago

@sergiocorato potresti riavviare il merge?

francesco-ooops commented 2 months ago

/ocabot merge minor

OCA-git-bot commented 2 months ago

Hey, thanks for contributing! Proceeding to merge this for you. Prepared branch 14.0-ocabot-merge-pr-4113-by-francesco-ooops-bump-minor, awaiting test results.

OCA-git-bot commented 2 months ago

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