OCA / operating-unit

Manage Operating Units in Odoo
GNU Affero General Public License v3.0
72 stars 237 forks source link

[15.0][MIG] hr_expense_operating_unit #574

Closed Saran440 closed 1 year ago

Saran440 commented 1 year ago

This PR continue from https://github.com/OCA/operating-unit/pull/449

JuanyDForgeflow commented 1 year ago

@Saran440 I was testing with admin user who has access to all operating units

image image

But in only allows me to select the main one

image

If I change the default to another ou (Ex: B2C) it's selected on creation but the domain only shows the main ou on the selection.

image

Saran440 commented 1 year ago

@JuanyDForgeflow Thank you for your review. I fixed it.

Saran440 commented 1 year ago

Last commit fix bug can't register payment with config self-balance and this PR will depend on https://github.com/OCA/operating-unit/pull/581

AaronHForgeFlow commented 1 year ago

/ocabot merge nobump

OCA-git-bot commented 1 year 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). 🤖

OCA-git-bot commented 1 year ago

What a great day to merge this nice PR. Let's do it! Prepared branch 15.0-ocabot-merge-pr-574-by-AaronHForgeFlow-bump-nobump, awaiting test results.

OCA-git-bot commented 1 year ago

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