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

[IMP] stock_storage_type: Allow to re-apply putaway rules on computed location for a selected move line #938

Open
wants to merge 4 commits into
base: 16.0
Choose a base branch
from

Conversation

rousseldenis
Copy link
Contributor

@rousseldenis rousseldenis commented Oct 17, 2024

We must ensure the same destination is computed at each time.

During recomputation, this:

  • Allows to keep the same destination if it is still allowed

  • Allows to change the destination if it is not allowed anymore

  • Improve the writing of domains appending with AND()

@OCA-git-bot
Copy link
Contributor

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

@rousseldenis rousseldenis added enhancement New feature or request needs review labels Oct 17, 2024
@rousseldenis rousseldenis added this to the 16.0 milestone Oct 17, 2024
@rousseldenis
Copy link
Contributor Author

@jbaudoux @lmignon @phschmidt

@rousseldenis rousseldenis force-pushed the 16.0-imp-storage-type-dro branch from b734827 to 12e16f3 Compare October 18, 2024 06:51
Copy link

There hasn't been any activity on this pull request in the past 4 months, so it has been marked as stale and it will be closed automatically if no further activity occurs in the next 30 days.
If you want this PR to never become stale, please ask a PSC member to apply the "no stale" label.

@github-actions github-actions bot added the stale PR/Issue without recent activity, it'll be soon closed automatically. label Feb 16, 2025
@rousseldenis rousseldenis force-pushed the 16.0-imp-storage-type-dro branch from 5fe8e98 to 4365383 Compare February 18, 2025 17:42
… been populated

In the case of no product mixing, if a destination location has already been computed and
in the meantime, the location has been populated by another product, the computation
still propose the same location.

Exclude locations in that case.
@rousseldenis rousseldenis force-pushed the 16.0-imp-storage-type-dro branch from 4365383 to 1822824 Compare February 18, 2025 18:23
@rousseldenis rousseldenis added no stale Use this label to prevent the automated stale action from closing this PR/Issue. and removed stale PR/Issue without recent activity, it'll be soon closed automatically. labels Feb 19, 2025
…uantity

The current implementation was triggerring queries (read_group). Rely now
on ORM to get records as 'quant_ids' field exists on locations
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request needs review no stale Use this label to prevent the automated stale action from closing this PR/Issue.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants