Skip to content

Conversation

@AungKoKoLin1997
Copy link
Contributor

This module checks the variance between the purchase price and the product's standard price at the time of receipt picking validation. If the variance exceeds a given threshold, the notification can be left in the chatter. Additionally, an error can be triggered before receiving the stock if needed.

@qrtl QT4947

Copy link

@nobuQuartile nobuQuartile left a comment

Choose a reason for hiding this comment

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

LGTM
Code review and Functional review

Copy link
Member

@yostashiro yostashiro left a comment

Choose a reason for hiding this comment

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

Just very minor non-blocking points. LGTM.

@OCA-git-bot
Copy link
Contributor

This PR has the approved label and has been created more than 5 days ago. It should therefore be ready to merge by a maintainer (or a PSC member if the concerned addon has no declared maintainer). 🤖

@AungKoKoLin1997 AungKoKoLin1997 force-pushed the 16.0-add-purchase_stock_price_variance branch 3 times, most recently from 3ed70a7 to 8469f13 Compare March 13, 2025 08:29
@AungKoKoLin1997 AungKoKoLin1997 force-pushed the 16.0-add-purchase_stock_price_variance branch from 8469f13 to 19dff71 Compare May 27, 2025 07:43
@AungKoKoLin1997
Copy link
Contributor Author

@oca/stock-logistics-workflow-maintainers
Is it possible to look at this PR?

@rousseldenis rousseldenis added this to the 16.0 milestone May 27, 2025
@AungKoKoLin1997 AungKoKoLin1997 force-pushed the 16.0-add-purchase_stock_price_variance branch from 19dff71 to c2d54ea Compare July 11, 2025 02:22
@github-actions
Copy link

github-actions bot commented Nov 9, 2025

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 9, 2025
@AungKoKoLin1997 AungKoKoLin1997 force-pushed the 16.0-add-purchase_stock_price_variance branch from c2d54ea to 65b3676 Compare November 10, 2025 01:26
Copy link

@kanda999 kanda999 left a comment

Choose a reason for hiding this comment

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

LGTM: it works as expected

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved ready to merge 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.

6 participants