Skip to content

Conversation

@SirPyTech
Copy link

Steps:

  1. Create a Manager of Operating Units
  2. Create some Operating Units
  3. Log in as the created Manager
  4. Create an Invoice
  5. Click on the Operating Unit field

Before this change:
Not all Operating Units can be selected.

After this change:
All Operating Units can be selected.

@SirPyTech SirPyTech force-pushed the 14.0-fix-operating_unit-assign_to_manager branch from 4e1698e to 55d63a1 Compare December 31, 2025 13:49
Steps:
1. Create a Manager of Operating Units
2. Create some Operating Units
3. Log in as the created Manager
4. Create an Invoice
5. Click on the Operating Unit field

Before this change:
Not all Operating Units can be selected.

After this change:
All Operating Units can be selected.
@SirPyTech SirPyTech force-pushed the 14.0-fix-operating_unit-assign_to_manager branch from 55d63a1 to 8b23d50 Compare December 31, 2025 14:00
@SirPyTech SirPyTech marked this pull request as ready for review December 31, 2025 14:05
@StefanRijnhart StefanRijnhart changed the title [FIX] account_operating_unit: Manager can select all OUs [14.0][FIX] account_operating_unit: Manager can select all OUs Jan 5, 2026
Copy link

@Marco-ooops Marco-ooops left a comment

Choose a reason for hiding this comment

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

Test funzionale ok

Copy link
Contributor

@HekkiMelody HekkiMelody left a comment

Choose a reason for hiding this comment

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

Code review: LGTM

@OCA-git-bot
Copy link
Contributor

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
Copy link
Contributor

@AaronHForgeFlow what do you think? Code and functional approved, we can fw-port

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants