Skip to content

Conversation

@BhaveshHeliconia
Copy link
Contributor

@BhaveshHeliconia BhaveshHeliconia commented Mar 11, 2025

No description provided.

@BhaveshHeliconia BhaveshHeliconia force-pushed the 18.0-mig-purchase_stock_operating_unit branch from c2cf774 to b4866a1 Compare March 11, 2025 13:13
@BhaveshHeliconia BhaveshHeliconia mentioned this pull request Mar 12, 2025
22 tasks
Copy link

@kanisorn-kate kanisorn-kate left a comment

Choose a reason for hiding this comment

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

LGTM

@BhaveshHeliconia BhaveshHeliconia force-pushed the 18.0-mig-purchase_stock_operating_unit branch 2 times, most recently from 4c4cbe6 to 39e1334 Compare July 10, 2025 10:23
class PurchaseOrder(models.Model):
_inherit = "purchase.order"

picking_type_id = fields.Many2one(
Copy link
Member

Choose a reason for hiding this comment

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

Functional Test 👍

But I'm not sure. Should we use onchange instead compute with picking_type_id and operating_unit_id?

Copy link
Contributor

Choose a reason for hiding this comment

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

I think onchange makes more sense. Maybe a constraint if there is inconsistency with the operating unit.

@github-actions
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 Nov 16, 2025
@BhaveshHeliconia BhaveshHeliconia force-pushed the 18.0-mig-purchase_stock_operating_unit branch from 39e1334 to 5585f0d Compare November 17, 2025 08:18
@github-actions github-actions bot removed the stale PR/Issue without recent activity, it'll be soon closed automatically. label Nov 23, 2025
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.

5 participants