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

[10.0][IMP] stock_lot_sale_tracking: Add tracking details #760

Closed

Conversation

rousseldenis
Copy link
Sponsor Contributor

@rousseldenis rousseldenis commented Nov 15, 2019

When tracking delivery lots is important, it's quite useful to retrieve delivered ones from sale order.

This adds :

  • A button link on Sale Order level
  • A report entry menu in Inventory

image

image

image

@api.depends('sale_lot_tracking_ids')
def _compute_sale_lot_tracking_count(self):
for sale in self:
sale.sale_lot_tracking_count = len(sale.sale_lot_tracking_ids)
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 it would be better to use readgroup....

Copy link

@Cedric-Pigeon Cedric-Pigeon left a comment

Choose a reason for hiding this comment

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

Code review

@rousseldenis rousseldenis force-pushed the 10.0-imp-stock_lot_sale_tracking-dro branch from bd5a0bc to f491ec7 Compare December 10, 2019 08:59
@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 Mar 13, 2022
@github-actions github-actions bot closed this Apr 17, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement needs review stale PR/Issue without recent activity, it'll be soon closed automatically.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants