Skip to content

Conversation

@rousseldenis
Copy link
Contributor

No description provided.

@rousseldenis rousseldenis added this to the 16.0 milestone Apr 4, 2023
@rousseldenis rousseldenis marked this pull request as draft April 4, 2023 07:15
@rousseldenis
Copy link
Contributor Author

rousseldenis commented Apr 4, 2023

This maybe needs a module to implement a search on move_ids_without_package field as that one is needed in depends() to make working the client side without saving the picking and generates a warning in Odoo logs

@rousseldenis rousseldenis force-pushed the 16.0-add-stock-picking-is-ready-dro branch from 7242a4d to 6d7248e Compare April 4, 2023 07:27
@rousseldenis rousseldenis force-pushed the 16.0-add-stock-picking-is-ready-dro branch from 6d7248e to 8cc297a Compare April 4, 2023 09:44
@lmignon
Copy link
Contributor

lmignon commented Apr 5, 2023

@rousseldenis The concept is useful but I'm a little bit afraid by the performance cost on large pickings since on every operation, the system will need to reload all the operations to compute the field. We'll see how it works on our DB...

@rousseldenis
Copy link
Contributor Author

@rousseldenis The concept is useful but I'm a little bit afraid by the performance cost on large pickings since on every operation, the system will need to reload all the operations to compute the field. We'll see how it works on our DB...

Yes, I had that in mind too. But, I think this should be correct as it has no impact on operations that can be done outside the interface if the field is not loaded and in the interface, we are limited to the first 80 move lines.

Let's go testing

@github-actions
Copy link

github-actions bot commented Sep 3, 2023

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 Sep 3, 2023
@github-actions github-actions bot closed this Oct 8, 2023
@rousseldenis rousseldenis reopened this Oct 9, 2023
@rousseldenis rousseldenis marked this pull request as ready for review October 9, 2023 07:49
FranzPoize pushed a commit to akretion/stock-logistics-workflow that referenced this pull request Oct 18, 2023
Signed-off-by rousseldenis
@github-actions github-actions bot closed this Dec 3, 2023
@rousseldenis rousseldenis reopened this Dec 3, 2023
@github-actions github-actions bot closed this Jan 7, 2024
@rousseldenis rousseldenis reopened this Jan 8, 2024
@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 Jan 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

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.

2 participants