Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[16.0][IMP] stock_quantity_history_location: add usage view #351

Open
wants to merge 1 commit into
base: 16.0
Choose a base branch
from

Conversation

lfreeke
Copy link

@lfreeke lfreeke commented Nov 10, 2024

Let the user select usage (location type) 'view' to select all child location.
This is useful if you have a location that has a direct parent to the usage (location type) 'view'.

@OCA-git-bot
Copy link
Contributor

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

@rousseldenis
Copy link
Contributor

Hi @lfreeke thanks for this. Seems legit.

Could you check why tests are failing ?

@thomaspaulb
Copy link

/ocabot rebase

@OCA-git-bot
Copy link
Contributor

Sorry @thomaspaulb you are not allowed to rebase.

To do so you must either have push permissions on the repository, or be a declared maintainer of all modified addons.

If you wish to adopt an addon and become it's maintainer, open a pull request to add your GitHub login to the maintainers key of its manifest.

@lfreeke lfreeke force-pushed the 16.0-imp-stock_quantity_history_location_add_view_location branch from 311ffbc to 42aa093 Compare November 19, 2024 16:27
@lfreeke
Copy link
Author

lfreeke commented Nov 19, 2024

Hi @lfreeke thanks for this. Seems legit.

Could you check why tests are failing ?

I retriggerd the build and now everything is green

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.

4 participants