search

OCA / account-budgeting

38 stars 113 forks source link

[14.0][MIG] account_budget_oca #50

Closed Freni-OSI closed 3 years ago

patrickrwilson commented 3 years ago

@Freni-OSI can you squash all the automated commits ([UPD], Translations and .pot). to cleanup the commit history.

Freni-OSI commented 3 years ago

@Freni-OSI can you squash all the automated commits ([UPD], Translations and .pot). to cleanup the commit history.

It's Done.

patrickrwilson commented 3 years ago

@osi-scampbell @smangukiya can you help review this?

OCA-git-bot commented 3 years 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). 🤖

patrickrwilson commented 3 years ago

@Freni-OSI Please see @smangukiya comment, I've confirmed this happens in community also. I'm not exactly sure why but can you take a look please, perhaps there is a access group change.

Freni-OSI commented 3 years ago

@Freni-OSI Please see @smangukiya comment, I've confirmed this happens in community also. I'm not exactly sure why but can you take a look please, perhaps there is a access group change.

Hi, for that, the user has to enable - "Show Full Accounting Features" group. (as this access right is added in the security file) CC: @smangukiya @patrickrwilson

patrickrwilson commented 3 years ago

Hi, for that, the user has to enable - "Show Full Accounting Features" group. (as this access right is added in the security file)

Yep, that was it, thanks @Freni-OSI. This should be good to go now.

smangukiya commented 3 years ago

@Freni-OSI Please see @smangukiya comment, I've confirmed this happens in community also. I'm not exactly sure why but can you take a look please, perhaps there is a access group change.

Hi, for that, the user has to enable - "Show Full Accounting Features" group. (as this access right is added in the security file) CC: @smangukiya @patrickrwilson

@Freni-OSI : Thank you for your contribution!

max3903 commented 3 years ago

/ocabot merge nobump

OCA-git-bot commented 3 years ago

This PR looks fantastic, let's merge it! Prepared branch 14.0-ocabot-merge-pr-50-by-max3903-bump-nobump, awaiting test results.

OCA-git-bot commented 3 years ago

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