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

[18.0][ADD] sale_automatic_workflow_stock_job #5

Open
wants to merge 2 commits into
base: 18.0
Choose a base branch
from

Conversation

chaule97
Copy link
Owner

@chaule97 chaule97 commented Dec 4, 2024

  • Because this commit separated the module stock from sale_automatic_workflow. I add new module sale_automatic_workflow_stock_job which includes the feature related to job of stock.

@api.depends("delivery_status")
def _compute_all_qty_delivered(self):
for order in self:
order.all_qty_delivered = order.delivery_status == "full"

Choose a reason for hiding this comment

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

this should be in sale_automatic_workflow_stock instead of this module

@api.onchange("workflow_process_id")
def _onchange_workflow_process_id(self):
if self.workflow_process_id.picking_policy:
self.picking_policy = self.workflow_process_id.picking_policy

Choose a reason for hiding this comment

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

same here, should be in sale_automatic_workflow_stock instead of this module

)

@api.model
def _handle_pickings(self, sale_workflow):

Choose a reason for hiding this comment

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

has been handled in sale_automatic_workflow_stock

@chaule97 chaule97 force-pushed the 18.0-add-sale_automatic_workflow_stock_job branch from a5ded86 to 4ee17a9 Compare December 5, 2024 04:49
@chaule97 chaule97 force-pushed the 18.0-add-sale_automatic_workflow_stock_job branch from 4ee17a9 to 43f4626 Compare December 5, 2024 10:50
@chaule97 chaule97 force-pushed the 18.0-add-sale_automatic_workflow_stock_job branch 4 times, most recently from f079069 to a5197c9 Compare January 13, 2025 02:46
@chaule97 chaule97 force-pushed the 18.0-add-sale_automatic_workflow_stock_job branch from a5197c9 to bc78cc4 Compare January 21, 2025 08:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants